Skip to content

Rework script to run automated tests #1137

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.

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

Merged
merged 7 commits into from
Aug 4, 2022
Merged

Conversation

christianlupus
Copy link
Collaborator

This change will rework the current test framework with a more user-friendly one. Some bugs have been ironed out as well as some bad architectural decisions.

Closes #1131
Closes #824

This is a squash of multiple commits.

Created some docs

Added documentation on fixtures

Docuemntation of fictures

Adding more documentation

Signed-off-by: Christian Wolf <[email protected]>
@christianlupus christianlupus added meta and development Any optimization for the build process maintenance labels Aug 3, 2022
@christianlupus christianlupus self-assigned this Aug 3, 2022
@github-actions
Copy link

github-actions bot commented Aug 3, 2022

Unit Test Results

     33 files       33 suites   6m 57s ⏱️
   403 tests    403 ✔️ 0 💤 0
4 433 runs  4 433 ✔️ 0 💤 0

Results for commit 2607f14.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Aug 4, 2022

Codecov Report

Merging #1137 (2607f14) into master (3e70f56) will decrease coverage by 0.21%.
The diff coverage is 77.77%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1137      +/-   ##
============================================
- Coverage     57.89%   57.68%   -0.22%     
- Complexity        0      735     +735     
============================================
  Files            73       73              
  Lines          2292     2290       -2     
============================================
- Hits           1327     1321       -6     
- Misses          965      969       +4     
Flag Coverage Δ
integration 1.79% <5.55%> (-0.18%) ⬇️
migration 4.06% <0.00%> (+<0.01%) ⬆️
unittests 51.83% <72.22%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
lib/Exception/ImportException.php 0.00% <0.00%> (ø)
lib/Exception/InvalidJSONFileException.php 0.00% <0.00%> (ø)
lib/Exception/RecipeExistsException.php 0.00% <0.00%> (ø)
lib/Helper/UserFolderHelper.php 100.00% <ø> (ø)
lib/Service/DbCacheService.php 0.00% <0.00%> (ø)
lib/Exception/CouldNotGuessEncodingException.php 100.00% <100.00%> (ø)
lib/Exception/HtmlParsingException.php 100.00% <100.00%> (ø)
lib/Exception/InvalidDurationException.php 100.00% <100.00%> (ø)
lib/Exception/InvalidRecipeException.php 100.00% <100.00%> (ø)
lib/Exception/InvalidThumbnailTypeException.php 100.00% <100.00%> (ø)
... and 12 more

Created basic structure

Create argument parser

Further code to alter docker copntainers

Created basic structure to create env dumps

Adding latest code

Created basic fixture possible

Continued to work on scripts

Cleaned up a bit

Make code more separated

Refoctored a bit

Generally make unit tests run

Make running work in general

Make tabulate only optional

Enhancing interface

Enable docker in containers

Fix generation of code coverage reports

Allow access to DB by tests

Signed-off-by: Christian Wolf <[email protected]>

Fix bug in python code

Speed up mysql-based tests with eatmydata

Activate postgres sync mode as well

Restore complete subfixture upon activation

Registering new helper with GH actions

Signed-off-by: Christian Wolf <[email protected]>
Signed-off-by: Christian Wolf <[email protected]>

# Conflicts:
#	.github/actions/run-tests/reset-from-container.sh
Increased speed of integration tests

Make tests run with syncs as well

Signed-off-by: Christian Wolf <[email protected]>
Signed-off-by: Christian Wolf <[email protected]>
@christianlupus christianlupus force-pushed the dev/run-locally-rework branch from 9722410 to 5959442 Compare August 4, 2022 19:51
Signed-off-by: Christian Wolf <[email protected]>
@christianlupus christianlupus added this to the Release 0.9.14 milestone Aug 4, 2022
@christianlupus christianlupus marked this pull request as ready for review August 4, 2022 20:07
@christianlupus christianlupus merged commit 54b6b5a into master Aug 4, 2022
@christianlupus christianlupus deleted the dev/run-locally-rework branch August 4, 2022 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance meta and development Any optimization for the build process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the CI tests with master non-mandatory test Add superscript for run-locally.sh
1 participant