Skip to content

Fix unit tests

ltngames requested to merge fix-unit-tests into master

Description

Unit tests should have been a mandatory thing before I even released FeniXCLI, while I did have tests I had plenty of issues I feel I did not want to figure out. Well, it's never too late I guess. This MR is to fix all unit tests so that we have a base set of test to run in the CI and for each MR. Future fixes will benefit from this and should improve the stability of FeniXCLI.

  • All unit tests will be re-written to be CI environment friendly.
  • All tests will have to pass 100% before a merge is possible.
  • Some modules don't throw errors correctly this causes AVA unit tests to fail, these modules will need to be updated.

Related Issue

#36 (closed) will not be closed it goes much deeper than the basic updates provided by this merge request. The updates will ensure all tests pass and fix the few functions that are tested for this bug.

Also inspired and split from !26 (merged)

Motivation and Context

Automation of unit tests so merge requests are tested prior to merge. Also inspired from !26 (merged)

How Has This Been Tested?

Can you test unit tests? 🤣

Screenshots (if appropriate):

None Available

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Update all unit tests

Checklist:

  • My code follows the code style of this project.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • Update package.json scripts for lint
  • Update gitlab-ci to include test stage(s)
  • Ensure all tests pass via GitLab's CI
Edited by ltngames

Merge request reports