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.

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

Adding testing tags #4140

Merged
merged 28 commits into from
Feb 22, 2022
Merged

Adding testing tags #4140

merged 28 commits into from
Feb 22, 2022

Conversation

Mitch-Budibase
Copy link
Contributor

Current tags are 'smoke' and 'all'

Budibase Staging Release Bot and others added 5 commits December 16, 2021 10:28
@github-actions
Copy link
Contributor

github-actions bot commented Jan 21, 2022

CLA Assistant Lite bot:

Thank you for your submission - we really appreciate it ❤️. Like many open-source projects, we ask that you all sign a Contributor License Agreement before we can accept your contribution.

You can sign the CLA by just posting a Pull Request Comment, the same as the text below.

If you are contributing on behalf of a company, your company should contact us to sign a Corporate Contributor License Agreement, via community@budibase.com.


I have read the CLA Document and I hereby sign the CLA


3 out of 4 committers have signed the CLA.
@mike12345567
@Mitch-Budibase
@shogunpurple
@Budibase Staging Release Bot
Budibase Staging Release Bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request

Mitch-Budibase and others added 7 commits January 21, 2022 13:12
-Updating Commands file to include all recent changes
-Added query files (2 of them) - required for query level transformers testing
-Cypress.json updated to remove "/builder/"
-Updating cypress version in package.json
-Missing }) within addmultioptiondatatype.spec.js
createTable
-Pagination is now Test Env only

oracle
-Test Env only
-Removed smoke test tag

commands
-Fix for createApp command
-Increase wait for createRestQuery command
queryLevelTransformerFunction
-adding eslint-disable-next-line

queryLevelTransformerFunctionWithData
-adding eslint-disable-next-line
-Removing const stateCode - not required
ignoring lint for entire query support files, rather than just ignoring a specific line
@codecov-commenter
Copy link

codecov-commenter commented Jan 21, 2022

Codecov Report

Merging #4140 (35ea3f4) into develop (c7877af) will increase coverage by 0.02%.
The diff coverage is 50.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #4140      +/-   ##
===========================================
+ Coverage    67.89%   67.92%   +0.02%     
===========================================
  Files          144      144              
  Lines         4922     4923       +1     
  Branches       761      761              
===========================================
+ Hits          3342     3344       +2     
  Misses        1105     1105              
+ Partials       475      474       -1     
Impacted Files Coverage Δ
...s/server/src/api/controllers/row/internalSearch.js 5.26% <0.00%> (ø)
...ges/server/src/api/controllers/query/validation.js 100.00% <100.00%> (ø)
packages/server/src/middleware/joi-validator.js 76.47% <0.00%> (ø)
packages/server/src/api/controllers/query/index.js 75.78% <0.00%> (+1.05%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 59a2e0c...35ea3f4. Read the comment docs.

increasing wait time for createAutogeneratedScreens command
-associated with auto screens tests
Mitch-Budibase and others added 13 commits February 9, 2022 12:56
Many tests are failing against the CI
-I'm stripping a lot out for now to verify the CI going green
Temporary removal of radio buttons tests
I had removed a lot of these for checking against the CI - adding them back in
The run command now includes:

--headed
--browser chrome
General test fixes/updates

-Removed createUser.spec.js - Covered by another file
-Skiipping two tests within queryLevelTransformer
Including query level transformer specific files within eslintignore & prettierignore
datasourceWizard
-Making test env only based on datasource usage

revertApp
-Updating a click function
@Mitch-Budibase Mitch-Budibase merged commit b6bc10f into develop Feb 22, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Feb 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants