-
-
Notifications
You must be signed in to change notification settings - Fork 195
Fix tests #698
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
Open
rsanzante
wants to merge
16
commits into
jhedstrom:main
Choose a base branch
from
rsanzante:fix-tests-polished
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Fix tests #698
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add debug code Command to find behat.ymll file Remove file to see if there are more strange errors Remove codde that accessed deleted file Try another feature file Recover feature file that seems ok More debugging because it fails in GitHub but not in local Try to sanitize file Fix docker command More debugging Use the id used by container Debugging More debugging Aggressive sanitisation Use 1001 use for all docker commands and try to fix mail feature Remove 1001 because previous errors Test without sanitisation It's weird, now languagefeature seems to work, but mail not Focus in one single file Fix debug code Remove tag to see if there is any difference Try to triggert GH actions REmove progeress flag to see what is really doing Revert "Focus in one single file" This reverts commit f45bf3f. More debug Even more debugging Force config file path Force tag in command Remove composer.lock debugging and opiton so behat shows all run tests Remove Spanish file to see if it removes Parser issue dbg Try to fix "fatal: detected dubious ownership in repository at '/var/www/html'" Try again to use the right user when running docker commands test Another try
Right root user whici compsoer
try dbg dng2
forgot sudo Remove faulty step and see twhat happens with next steps Remove faulty feature Remove another faulty feature Remove another faulty feature Remove another faulty feature Remove another faulty feature Remove another faulty feature Revert "Remove faulty feature" This reverts commit f886185. Trigger tests Replace faulty feature with simplest one Dbg Display all test steps Add balckbox to test to avoid running it in d10 to see if language change is breaking tests ADd antoher blackbox tag to a translated test dbg Maybe files with same name and different lang is bad Move tag to scenario instead of feature Remove feature to see if it breaks the next one Remove cat Try double run Remove all Spanish tests Revert "Remove all Spanish tests" This reverts commit 0e2d544. Check blackbox again Clear Behat/Gherkin cache There is no blackbox profile, it should be default suite FOrece en in behat conf
Move step that checkes features when they are ready Move command that also uses features foldder dbg Make me a sandwich Remove dobule run
It should but it seems it doesn't See gherkin-lint/gherkin-lint#265
Remove || true that always made tests pass
Double run
Remove debug code
The probem is the step of running cron usually happens when it is still running because previos actions, so let's wait for it.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes [#697].