Docker

[tauqirsarwar1]

To be filled by the PR creator:

• A brief description of the changes made -

• Do we have clean latest run report attached with this PR?

  • Yes
  • No (Please explain why)

• Does the PR contain changes to any BP core file?

  • Yes (Needs approval from at least 2 people)
  • No

• Does the PR contain changes to modules shared with other teams?

  • Yes (Needs approval from at least one of the other teams that use the module)
  • No

• Is it

  • New Testcase
  • Fix

To be filled by the PR reviewer:

• Verify the attached run report passed in GitHub Actions (Justify if local run)

• General

  • Use the best strategy to locate the elements
  • Comments wherever the code is not readable by itself
  • Use of the right data structure for the use case
  • Reuse logic/functionality as much as possible
  • Cleanup of any test data that is generated by the tests
  • No static waits