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

166 add unit tests #199

Merged
merged 82 commits into from
Dec 20, 2024
Merged

166 add unit tests #199

merged 82 commits into from
Dec 20, 2024

Conversation

JavierCladellas
Copy link
Collaborator

Wrote various test suites for application modules. Currently 18 test files, with a total of 287 (parametrized) tests.
Fixed some bugs that raised while testing.
Added unit tests to the CI pipeline

Still pending:

  • Repository tests (reports)
  • Component tests (reports)
  • MVC tests (reports)
  • Script tests
  • figureFactory tests

@JavierCladellas
start validation tests #166
f66d1f9
@JavierCladellas
bug fix with logspace param (missing else)
8b77057
@JavierCladellas
logspace -> geomspace
721a570
@JavierCladellas
rm base from logspace
973c14c
@JavierCladellas
add geomspace validation
e22ca9c
@JavierCladellas
fix params
8427afc
@JavierCladellas
forgot return self :/
552c1de
@JavierCladellas
fix object casting
e9c277c
@JavierCladellas
start parameter tests #166
73961a0
@JavierCladellas
add pytest to ci #166
efa41b4
@JavierCladellas
split parameter tests #166
31bb284
@JavierCladellas
add more parameter test cases #166
e22d0e9
@JavierCladellas
added tests for zip #166
7c4b793
@JavierCladellas
mv sanity validationHandler test #166
78b4bd7
@JavierCladellas
rename validationHandler
dc5ea1b
@JavierCladellas
docstrings #166
84b3aca
@JavierCladellas
docstrings #166
c28cb52
@JavierCladellas
docstrings
5272e34
@JavierCladellas
add n_steps validation
742113c
@JavierCladellas
add range validation
bc3d69a
@JavierCladellas
rm test case bc of validation #166
1c1086a
@JavierCladellas
small refactor
5aca8b5
@JavierCladellas
wrote parameter validation unit tests #166
b40210f
@JavierCladellas
docstring
7ed1e39
@JavierCladellas
add default report dir path
ea2a395
@JavierCladellas
add mock app config
ad46034
@JavierCladellas
start reframe parameter tests (hard) :`( #166
dfe18ba
@JavierCladellas
rename fct & fix random rest #166
259f3a3
@JavierCladellas
mv CommandBuilder
2961819
@JavierCladellas
fix datetime
245b589
@JavierCladellas JavierCladellas added enhancement A new feature or request step: check check step ci-cd Continuous Integration/Continuous Deployment labels Dec 17, 2024
@JavierCladellas JavierCladellas self-assigned this Dec 17, 2024
@JavierCladellas JavierCladellas linked an issue Dec 17, 2024 that may be closed by this pull request
Add unit tests #166
Closed
@netlify Netlify
Copy link

netlify bot commented Dec 17, 2024 •
edited
Loading

✅ Deploy Preview for benchmarking-polite-crostata-92f389 ready!

Name Link
🔨 Latest commit f0affb0
🔍 Latest deploy log https://app.netlify.com/sites/benchmarking-polite-crostata-92f389/deploys/6765422e030a6b00083c1537
😎 Deploy Preview https://deploy-preview-199--benchmarking-polite-crostata-92f389.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JavierCladellas JavierCladellas merged commit df85dd5 into master Dec 20, 2024
10 checks passed
@JavierCladellas JavierCladellas deleted the 166-add-unit-tests branch December 20, 2024 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vincentchabannes vincentchabannes vincentchabannes approved these changes

@prudhomm prudhomm Awaiting requested review from prudhomm

Assignees

@JavierCladellas JavierCladellas

Labels
ci-cd Continuous Integration/Continuous Deployment enhancement A new feature or request step: check check step
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add unit tests
2 participants
@JavierCladellas @vincentchabannes