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/18 refactoring of response action to dynamic #26

Merged

Conversation

miroslavpojer
Copy link
Collaborator

@miroslavpojer miroslavpojer commented Aug 22, 2023

#18 - Refactoring of ResponseAction to dynamic

  • Refactored responseActions logic to accept random parameter names.
  • Correct parameters are validated in time of project loading.
  • Extended json schemas by descriptions and version.

Closes #18

* Renamed assertion json tag to responseAction.
* Refactored related logic and doc.
* Simplified json files for ResponseAction - ResponseActions group and name by method with meaning "group.name". In code this value is separated for more intuitive usage.
* Refactored responseActions logic to accept random parameter names.
* Correct parameters are validated in time of project loading.
@miroslavpojer miroslavpojer self-assigned this Aug 22, 2023
@miroslavpojer miroslavpojer removed the request for review from Zejnilovic August 22, 2023 10:32
@github-actions
Copy link

github-actions bot commented Aug 22, 2023

JaCoCo code coverage report - ScAPI

File Coverage [89.05%] 🍏
AssertionResponseAction.scala 92.76% 🍏
LogResponseAction.scala 92% 🍏
ExtractJsonResponseAction.scala 91.64% 🍏
SuiteFactory.scala 85.55% 🍏
Total Project Coverage 84.02% 🍏

…o_dynamic

# Conflicts:
#	testApi/src/main/resources/schema/after.schema.json
#	testApi/src/main/resources/schema/before.schema.json
#	testApi/src/main/resources/schema/suite.schema.json
#	testApi/src/main/scala/africa/absa/testing/scapi/json/SuiteFactory.scala
#	testApi/src/main/scala/africa/absa/testing/scapi/rest/response/AssertionResponseAction.scala
#	testApi/src/main/scala/africa/absa/testing/scapi/rest/response/ExtractJsonResponseAction.scala
#	testApi/src/main/scala/africa/absa/testing/scapi/rest/response/LogResponseAction.scala
#	testApi/src/test/resources/project_with_issues/suites/aulgui-controller/undefinedConstantIssue.suite.json
#	testApi/src/test/resources/test_project/suites/gui-controller/deleteQuestion.suite.json
#	testApi/src/test/resources/test_project/suites/gui-controller/getUserCurrent.after.json
#	testApi/src/test/resources/test_project/suites/gui-controller/getUserCurrent.before.json
#	testApi/src/test/resources/test_project/suites/gui-controller/getUserCurrent.suite.json
#	testApi/src/test/resources/test_project/suites/gui-controller/postQuestion.suite.json
#	testApi/src/test/resources/test_project/suites/gui-controller/putQuestion.suite.json
#	testApi/src/test/scala/africa/absa/testing/scapi/rest/response/ResponseAssertionsTest.scala
#	testApi/src/test/scala/africa/absa/testing/scapi/rest/response/ResponseExtractTest.scala
#	testApi/src/test/scala/africa/absa/testing/scapi/rest/response/ResponseLogTest.scala
#	testApi/src/test/scala/africa/absa/testing/scapi/suite/runner/SuiteRunnerTest.scala
@miroslavpojer miroslavpojer merged commit bd0ad01 into master Aug 26, 2023
2 checks passed
@miroslavpojer miroslavpojer deleted the feature/18-Refactoring_of_ResponseAction_to_dynamic branch August 26, 2023 14:56
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.

Refactoring of ResponseAction to dynamic
2 participants