Skip to content

Conversation

@MartinSchoeler
Copy link
Member

No description provided.

Comment on lines +58 to +61
await test.step('Expect to be able to open canned responses sidebar and creation', async () => {
await agent.poHomeChannel.content.btnCannedResponses.click();
await agent.poHomeChannel.content.btnNewCannedResponse.click();
});
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't we have at least one assertion here to confirm if the screen is in the expected state?

Copy link
Member Author

Choose a reason for hiding this comment

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

The idea is to create the framework of the test so the coverage can detect the changes

@codecov
Copy link

codecov bot commented Aug 26, 2022

Codecov Report

Merging #26706 (655c3fc) into develop (0790192) will increase coverage by 0.28%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #26706      +/-   ##
===========================================
+ Coverage    40.39%   40.67%   +0.28%     
===========================================
  Files          802      800       -2     
  Lines        18446    18324     -122     
  Branches      1956     1956              
===========================================
+ Hits          7451     7454       +3     
+ Misses       10702    10578     -124     
+ Partials       293      292       -1     
Flag Coverage Δ
e2e 40.67% <ø> (+0.28%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@ggazzo ggazzo merged commit 36d5252 into develop Aug 26, 2022
@ggazzo ggazzo deleted the remove-canned-response-templates branch August 26, 2022 19:06
@murtaza98 murtaza98 mentioned this pull request Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove/Convert cannedResponses.html/js Remove/Convert departmentCannedResponses.html/js

3 participants