Skip to content
This repository has been archived by the owner on Aug 14, 2024. It is now read-only.

Revamp tests #13

Closed
Kawacrepe opened this issue Aug 13, 2022 · 5 comments
Closed

Revamp tests #13

Kawacrepe opened this issue Aug 13, 2022 · 5 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@Kawacrepe
Copy link
Member

Kawacrepe commented Aug 13, 2022

Our tests must be refacto, we must execute temp.js to create the nsecure-result.json which will be analyzed by the package.

We want to remove this file and include the step in our tests.

The second improvement consist in what Rossbob told me in another PR


I would have added some tests over rewriting it :

  • one without any options (flags & domainInformations)
  • one with flags
  • one with domainInformations

but it may be overkill

Originally posted by @Rossb0b in #12 (review)

@Kawacrepe Kawacrepe added the good first issue Good for newcomers label Aug 13, 2022
@Kawacrepe Kawacrepe reopened this Jan 19, 2023
@fabnguess fabnguess self-assigned this Aug 23, 2023
@fabnguess
Copy link
Contributor

@Kawacrepe can I take this issue?

@Kawacrepe
Copy link
Member Author

Yes, feel free to ping me if some additionnals informations are needed :)

@fabnguess
Copy link
Contributor

@Kawacrepe Yes, I want to make sure I understood the task correctly. This is effectively removing the temp.js file after carefully transposing its logic into a unit test. In conclusion, should we also implement Rossbob's recommendations

@Kawacrepe
Copy link
Member Author

Exactly, we should look at the rossbob recommendations :)

@fraxken
Copy link
Member

fraxken commented Sep 3, 2023

Going to rework them

@fraxken fraxken assigned fraxken and unassigned fabnguess Sep 3, 2023
@fraxken fraxken closed this as completed Jul 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue Good for newcomers
Projects
Archived in project
Development

No branches or pull requests

3 participants