Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New View Tests #62

Merged
merged 20 commits into from
Oct 18, 2017
Merged

New View Tests #62

merged 20 commits into from
Oct 18, 2017

Conversation

Awinja-j
Copy link
Contributor

@Awinja-j Awinja-j commented Oct 5, 2017

This PR Introduces tests files for the available views which are:

  • Doctors
  • Clinical officers
  • Nurses
  • Health Facilities
  • NHIF Inpatient
  • NHIF Outpatient
  • NHIF Outpatient Civil Servants

This task involved writing the tests and running the tests to ensure they run successfully.

This tests can be run by using the following commands:

nosetests healthtools/tests
nosetests healthtools/tests/doctor.py
nosetests healthtools/tests/nurse.py
nosetests healthtools/tests/clinical_officer.py
nosetests healthtools/tests/health_facilities.py
nosetests healthtools/tests/nhif_inpatient.py
nosetests healthtools/tests/nhif_outpatient.py
nosetests healthtools/tests/nhif_outpatient_cs.py

Relevant Github Issue:

#46

currentdir = os.path.dirname(os.path.abspath(
inspect.getfile(inspect.currentframe())))
parentdir = os.path.dirname(currentdir)
sys.path.insert(0, parentdir)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What are these four lines of code for?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Awinja-Andela ☝️

@DavidLemayian
Copy link
Contributor

This is the exact same PR I closed a couple of days ago. Did you open this one by mistake? #59 (comment)

@Awinja-j
Copy link
Contributor Author

Awinja-j commented Oct 7, 2017

@DavidLemayian This is a new repo. i added new test cases that i feel make use of the iterators in the documents.

@DavidLemayian
Copy link
Contributor

DavidLemayian commented Oct 16, 2017

Hi @Awinja-Andela. Going to re-open this as there are many ways to skin a cat.

@DavidLemayian DavidLemayian reopened this Oct 16, 2017
@Awinja-j
Copy link
Contributor Author

@DavidLemayian please check on changes i have made.

@DavidLemayian DavidLemayian merged commit 72e1a49 into CodeForAfrica:develop Oct 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants