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.

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

Angular unit tests #232

Open
therealryan opened this issue Feb 2, 2023 · 1 comment
Open

Angular unit tests #232

therealryan opened this issue Feb 2, 2023 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers javascript Pull requests that update Javascript code

Comments

@therealryan
Copy link
Contributor

therealryan commented Feb 2, 2023 •
edited
Loading

While we've got test coverage of the execution report behaviour on the java side, there are a whole bunch of empty angular unit tests that have been ignored since they were autogenerated (for example)

It'd be good to fill those in.

There are a couple of examples to draw on:

Test coverage is reported at the bottom of the build artifacts page, for example
@therealryan therealryan added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed javascript Pull requests that update Javascript code and removed help wanted Extra attention is needed labels Feb 2, 2023
@therealryan
Copy link
Contributor Author

therealryan commented Jun 1, 2023 •
edited
Loading

#399 takes a big step forwards for this effort - the tests all run without errors (they still don't really test anything though).

@therealryan therealryan mentioned this issue Oct 25, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers javascript Pull requests that update Javascript code
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@therealryan