Skip to content

Ci reference #514

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
Merged
merged 14 commits into from
Sep 14, 2021
Merged

Ci reference #514

merged 14 commits into from
Sep 14, 2021

Conversation

@duncdrum
Copy link
Contributor

@duncdrum duncdrum commented Sep 14, 2021 •
edited
Loading

Brings back specs 01-03. The test for the missing restxq item is skipped see #508

#513 is biting us on headless, but hence the failing 06 spec but now with proper error reporting, the DOM detachment can be avoided in headed mode, but this is still hacky. A proper fix requires a design discussion.

04 + 05 are skipped, to not waste CI cycles, getting them to pass is WIP see #459

close #485 #472 #512

duncdrum and others added 14 commits September 13, 2021 10:28
@duncdrum
chore(ignore): ignore .xar
e052ec2
@duncdrum
ref(api_spec): separate api calls from UI display
7168464 
see #399 #508
@duncdrum
test(rest): skip test calls to restxq
5fb0fb4 
see #508
@duncdrum
test(doc_spec): redirect ci to tree window again
e11d767 
see #413
@duncdrum
chore(deps): update cypress, mocha, ts-node
1e54af6 
remove unused drag-n-drop extension see #22
@ccheraa @duncdrum
removed old cypress hack
4f0d502
@ccheraa @duncdrum
fix: undefined process after moving to webpack 5
09bdf07
@duncdrum
ignore(api_spec): lint
f34dd5d
@duncdrum
chore(deps): bump cypress 8.4.0
0a00a61
@duncdrum
ci(e2e): switch ci to electron
7263fed 
see #452 #511
#id 158
@duncdrum
fix(eval_spec): css selector changes
ced7d94 
close #509
@duncdrum
perf(ci): remove browser deps
ed72601 
rebalance testsuite

see #452
#taskid 158
@duncdrum
test(ci): rebalance test
219e91e 
skip doc and collection test for faster feedback

make #508 harder to ignore
@duncdrum
fix(permission_spec): add more guards
2c2e723 
see  #397
the spec currently fails in headless mode but passes in headed mode

see #513
@duncdrum duncdrum added test Anything related to testing the extension High Priority High Priority expedite to unblock labels Sep 14, 2021
@duncdrum duncdrum requested a review from ccheraa September 14, 2021 14:35
@cla-bot cla-bot bot added the cla-signed label Sep 14, 2021
@duncdrum
Copy link
Contributor Author

@adamretter can you please remove the appveyor integrations, we'll move to windows testing in due time, but in their current state they are just noise.

@adamretter adamretter merged commit 14651f8 into master Sep 14, 2021
@duncdrum duncdrum deleted the ci-reference branch September 14, 2021 16:35
This was referenced Sep 15, 2021
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ccheraa ccheraa Awaiting requested review from ccheraa

Assignees

No one assigned

Labels

cla-signed High Priority High Priority expedite to unblock test Anything related to testing the extension

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@duncdrum @adamretter @ccheraa