Skip to content

🎨 refactor: Edited structure, code style, custom Allure reports#2

Open
SuperTapok wants to merge 3 commits into
masterfrom
develop
Open

🎨 refactor: Edited structure, code style, custom Allure reports#2
SuperTapok wants to merge 3 commits into
masterfrom
develop

Conversation

@SuperTapok

Copy link
Copy Markdown
Owner
  • Edited project structure - added constants interface, fixed typos
  • Added allure annotations for every test

* Edited project structure - added constants interface, fixed typos
* Added allure annotations for every test

@dalekskaro dalekskaro left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Самое главное из ревью: дополнить Settings.java

Comment thread README.md Outdated
Comment thread pom.xml
Comment thread src/main/java/constants/Constants.java
Comment thread src/main/java/constants/Constants.java Outdated
Comment thread src/main/java/constants/Constants.java Outdated
Arguments.of(16, "Romeo", new ArrayList<>(Arrays.asList("url1", "url2"))),
Arguments.of(17, "Julia", new ArrayList<>(Arrays.asList("url1", "url2"))),
Arguments.of(18, "Andrew", new ArrayList<>(Arrays.asList("url1", "url2")))
Arguments.of(15, "Bullet", new ArrayList<>(Arrays.asList("url1", "url2"))),

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Если совсем упарываться то все значения здесь надо вынести в отдельный интерфейс, но это если уж совсем не будет нечем занятсья

Comment thread src/main/java/helpers/Settings.java
Comment thread src/test/java/pet/PetNegativeTest.java Outdated
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.

2 participants