Skip to content

Need unit tests #1

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.

Sign up for GitHub

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

Jump to bottom
Closed
abmusse opened this issue Feb 20, 2017 · 4 comments
Closed

Need unit tests #1

abmusse opened this issue Feb 20, 2017 · 4 comments
Assignees
@abmusse
Labels
enhancement New feature or request
Milestone
Version 1.0

Comments

@abmusse
Copy link
Member

abmusse commented Feb 20, 2017

Original report by Aaron Bartell (Bitbucket: aaronbartell, GitHub: aaronbartell).

Need to create unit tests so we can be sure changes don't break other parts of the code base.
@abmusse
Copy link
Member Author

abmusse commented Feb 22, 2017

Original comment by Xu Meng (Bitbucket: mengxumx, GitHub: dmabupt).

I have added a basic test script in the test directory. It is based on Mocha.

@abmusse abmusse added major enhancement New feature or request labels Jan 22, 2019
@kadler kadler added this to the Version 1.0 milestone Jan 22, 2019
@abmusse
Copy link
Member Author

abmusse commented Jan 24, 2019

I am currently working on mocha unit tests for:

  • iConn Class
  • iDataQueue Class
  • iObj Class
  • iPgm Class
  • iSql Class
  • iUserSpace Class
  • iWork Class
  • iProd Class
  • iNetwork Class

@kadler kadler removed the major label Jan 24, 2019
@abmusse abmusse self-assigned this Feb 4, 2019
@abmusse
Copy link
Member Author

abmusse commented Feb 28, 2019

Kicked this off with PR #34.

Its a good start but some functional tests can be improved by not only checking the existence of properties but also validating the values.

In some cases the values are dynamic we should try to ensure it matches and expected pattern or explore other options.

@abmusse
Copy link
Member Author

abmusse commented Jun 13, 2019

Closing this because #34 adds unit and functional tests

@abmusse abmusse closed this as completed Jun 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abmusse abmusse

Labels
enhancement New feature or request
Projects
None yet
Milestone
Version 1.0
Development

No branches or pull requests
2 participants
@kadler @abmusse