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

NT-2124: Shot library integration #1324

Merged
merged 9 commits into from Jul 19, 2021
Merged

Conversation

Arkariang
Copy link
Contributor

@Arkariang Arkariang commented Jul 15, 2021

📲 What

  • Integration with Shot open source library.
  • Remove the previous existing flavor regression, it was using UIAutomator in order to do end - to - end tests. Those test are no longer used.

🤔 Why

  • We are using Shot screenshot library because it provides the ability to compare/record shreenshot tests.

🛠 How

  • Configured Shot plugin on build.gradle
  • Followed specifications from Shot

👀 See

Screen Shot 2021-07-15 at 2 47 07 PM

| | |

📋 QA

  • Checkout the branch, clean project and build project again, go to the test classCommentCardShotTest and execute it, you should see the same result as I showcase on the image up

Story 📖

NT-2124

@Arkariang Arkariang changed the title Imartin/nt 2124 integration shot NT-2124: Shot library integration Jul 15, 2021
@codecov
Copy link

codecov bot commented Jul 15, 2021

Codecov Report

Merging #1324 (0eeac9c) into master (1fb18b6) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1324   +/-   ##
=========================================
  Coverage     74.12%   74.12%           
  Complexity      739      739           
=========================================
  Files           221      221           
  Lines          6666     6666           
  Branches        406      406           
=========================================
  Hits           4941     4941           
  Misses         1589     1589           
  Partials        136      136           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1fb18b6...0eeac9c. Read the comment docs.

Copy link
Contributor

@hadia hadia left a comment

Choose a reason for hiding this comment

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

👌🎉

@Arkariang Arkariang merged commit 08c2c19 into master Jul 19, 2021
@Arkariang Arkariang deleted the imartin/NT-2124-integration-shot branch July 19, 2021 17:30
Arkariang added a commit that referenced this pull request Jul 19, 2021
…tin/NT-2125-first-test

* 'master' of github.com:kickstarter/android-oss:
  - Removed sharedUserId property
  NT-2003:UX – Show pending comment dialog (#1329)
  NT-1996 :UI/UX – Error loading thread (#1328)
  NT-2093: Skip first emitted value from paginated data to avoid initially showing empty state showing on root comments screen (#1323)
  NT-2124: Shot 5.11.0 library integration (#1324)

# Conflicts:
#	app/build.gradle
#	app/src/androidTest/java/com/kickstarter/screenshoot/testing/di/AndroidTestApplicationModule.kt
#	app/src/androidTest/java/com/kickstarter/screenshoot/testing/ui/components/CommentCardShotTest.kt
#	app/src/internalDebug/AndroidManifest.xml
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.

None yet

2 participants