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

feature: test as a resource #2701

Merged
merged 32 commits into from
Jul 6, 2023
Merged

feature: test as a resource #2701

merged 32 commits into from
Jul 6, 2023

Conversation

xoscar
Copy link
Contributor

@xoscar xoscar commented Jun 9, 2023

This PR changes Tracetest Frontend, CLI and Server to handle tests as a resource. With that, we have new commands to handle tests in Tracetest CLI and we simplified the middleware that we have to deal with tests.

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

@mathnogueira mathnogueira force-pushed the feat/tests-as-resources branch from c029ef2 to 374d920 Compare June 13, 2023 13:44
@mathnogueira
Copy link
Contributor

Rebased with main

@mathnogueira mathnogueira mentioned this pull request Jun 13, 2023
4 tasks
@mathnogueira mathnogueira force-pushed the feat/tests-as-resources branch from b8a40a4 to 2078253 Compare June 14, 2023 14:38
@schoren schoren force-pushed the feat/tests-as-resources branch 2 times, most recently from e6dcf65 to 924b9e5 Compare June 22, 2023 14:13
mathnogueira and others added 6 commits June 26, 2023 14:44
update get tests endpoint to use resource manager
* fix: move transactions to the transactions folder

* remove alias to transactions module
* add test, run, and trigger objects

* feat: move test and run structs into the test package

* list tests

* add old format attributes again

* fix mapping

* compatibility mode

* fix server prefix integration test

* force server unit tests to run

* fix tracetest test

* trigger CI

* Apply suggestions from code review

Co-authored-by: Sebastian Choren <[email protected]>

* fix build

* version test specs

* update test names

* use sqlutils

* dont specify branch target in pull_request

* fix list test

---------

Co-authored-by: Sebastian Choren <[email protected]>
…get tests frontend (#2702)

chore(frontend): updating FE to support GET /tests endpoint changes
@mathnogueira mathnogueira force-pushed the feat/tests-as-resources branch from ab7acbb to 40af921 Compare June 26, 2023 17:55
mathnogueira and others added 11 commits July 3, 2023 11:47
* chore(docs): Adding App Insights Configuration Page (#2820)

* chore(docs): Adding App Insights Configuration Page

* fixing typo

* fix(frontend): Fixing Resizable Panels UI bugs (#2827)

* fix(tests): add analyzer resource to cli e2e table (#2826)

* chore(docs): Azure App Insights Recipes (#2821)

* chore(docs): Adding App Insights Configuration Page

* chore(docs): Adding App Insights Recipes

* chore(docs): Adding App Insights Recipes

* chore(docs): fixing typo

* adding recipe links

* adding recipe links

* fixing typo

* chore(frotend): Test Definition Name Input (#2830)

* chore(frotend): Updating Panel Splitter w/ Tooltip (#2828)

* chore(frotend): Updating Panel Splitter w/ Tooltip

* Updating theme color

* removing unnecessary flag

* undo change

* fix(frontend): fix missing font-face (#2833)

* feat(cli): refactor list formatter for better resource manager support (#2829)

* feat(frontend): add independent trace vs test data (#2815)

* feat(cli): refactor Get formatter for better resource manager support (#2831)

* feat(cli): dynamic list of available resources (#2832)

* feat(cli): refactor Delete to new resource manager client (#2836)

* feat(cli): update export command with new resourcemanager client (#2838)

* Standardize old resources to to use list method with SQL Injection protection and stardardize PollingProfile file (#2839)

* wip: update list on demo and environment to use the same standards as tests and transactions

* Fixing provisioning test

* Update provisioning test

* feature(frontend): Contact Us Modal (#2835)

* feature(frontend): Contact Us Modal

* feature(frontend): Contact Us Modal

* feat(cli): update apply command with new resourcemanager client (#2841)

* feat(frontend): trace vs test data for timeline and attribute list (#2837)

* feat: new tests openapi spec (#2658)

update get tests endpoint to use resource manager

* fix: move transactions to it's own module (#2664)

* fix: move transactions to the transactions folder

* remove alias to transactions module

* feat: test models + list tests endpoint (#2681)

* add test, run, and trigger objects

* feat: move test and run structs into the test package

* list tests

* add old format attributes again

* fix mapping

* compatibility mode

* fix server prefix integration test

* force server unit tests to run

* fix tracetest test

* trigger CI

* Apply suggestions from code review

Co-authored-by: Sebastian Choren <[email protected]>

* fix build

* version test specs

* update test names

* use sqlutils

* dont specify branch target in pull_request

* fix list test

---------

Co-authored-by: Sebastian Choren <[email protected]>

* 2659 cli improvements migrate tests to resource manager architecture get tests frontend (#2702)

chore(frontend): updating FE to support GET /tests endpoint changes

* feat: test list augmented response (#2732)

add augmented list

* fix(server): fix trigger json encoding versioning and test (#2795)

* feat: test models + list tests endpoint (#2681)

* add test, run, and trigger objects

* feat: move test and run structs into the test package

* list tests

* add old format attributes again

* fix mapping

* compatibility mode

* fix server prefix integration test

* force server unit tests to run

* fix tracetest test

* trigger CI

* Apply suggestions from code review

Co-authored-by: Sebastian Choren <[email protected]>

* fix build

* version test specs

* update test names

* use sqlutils

* dont specify branch target in pull_request

* fix list test

---------

Co-authored-by: Sebastian Choren <[email protected]>

* GET endpoint

* fix rebase

* add create method and fix tests

* add rest of methods

* Changing test structure and fixing method by method

* Update tests

* Fixing test errors for Delete

* Adding gRPC tests

* Adding tests for traceid trigger

* migrate rest of endpoints and fix compilation errors

* fix all unit and integration tests

* Improving tests

* fix building errors

---------

Co-authored-by: Oscar Reyes <[email protected]>
Co-authored-by: Jorge Padilla <[email protected]>
Co-authored-by: Sebastian Choren <[email protected]>
Co-authored-by: Daniel Baptista Dias <[email protected]>
Co-authored-by: Daniel Dias <[email protected]>
fix grpc tests and transaction tracetests
@schoren schoren force-pushed the feat/tests-as-resources branch 2 times, most recently from 581325a to d3b5a45 Compare July 4, 2023 23:26
@danielbdias danielbdias changed the title Feat/tests as resources feature: test as a resource Jul 6, 2023
@danielbdias danielbdias marked this pull request as ready for review July 6, 2023 13:06
Copy link
Contributor

@mathnogueira mathnogueira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feels great to finish this body of work!

@xoscar xoscar merged commit bc05cd2 into main Jul 6, 2023
@xoscar xoscar deleted the feat/tests-as-resources branch July 6, 2023 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants