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

[Ep-458] QA Page Viewed (Project) Phase 1 #1424

Merged
merged 10 commits into from
Apr 13, 2021

Conversation

tobitech
Copy link
Contributor

📲 What

Page viewed for project was tracked in two places.
Moved the one for campaign section context into Project Description screen when the content loads.
Add RefTag as part of the Config Data sent to ProjectDescription screen.

🤔 Why

This is part of phase 1 Events QA.

👀 See

iOS-QA-Page-Viewed-(Project)-Phase-1

…scriptionViewModel

Include RefTag as part of configuration data for ProjectDescriptionViewController
Provide a helper test method to compare tuple values of ProjectPamphletMainCellData
@codecov
Copy link

codecov bot commented Apr 12, 2021

Codecov Report

Merging #1424 (eb7fd5e) into master (cb843e2) will increase coverage by 0.00%.
The diff coverage is 98.63%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1424   +/-   ##
=======================================
  Coverage   86.08%   86.09%           
=======================================
  Files        1106     1106           
  Lines       98101    98096    -5     
=======================================
+ Hits        84453    84455    +2     
+ Misses      13648    13641    -7     
Impacted Files Coverage Δ
...rollers/ProjectPamphletContentViewController.swift 46.15% <0.00%> (ø)
...kstarter-iOS/ViewModels/AppDelegateViewModel.swift 96.54% <100.00%> (+0.15%) ⬆️
...ter-iOS/ViewModels/AppDelegateViewModelTests.swift 99.65% <100.00%> (+<0.01%) ⬆️
...rter-iOS/Views/Cells/ProjectPamphletMainCell.swift 89.15% <100.00%> (ø)
...Controllers/ProjectDescriptionViewController.swift 8.00% <100.00%> (+8.00%) ⬆️
...brary/ViewModels/ProjectDescriptionViewModel.swift 100.00% <100.00%> (ø)
.../ViewModels/ProjectDescriptionViewModelTests.swift 100.00% <100.00%> (ø)
.../ViewModels/ProjectPamphletMainCellViewModel.swift 97.97% <100.00%> (-0.08%) ⬇️
...Models/ProjectPamphletMainCellViewModelTests.swift 100.00% <100.00%> (ø)
...ary/ViewModels/ProjectPamphletViewModelTests.swift 100.00% <100.00%> (ø)

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 cb843e2...eb7fd5e. Read the comment docs.

Copy link
Contributor

@singhhari singhhari left a comment

Choose a reason for hiding this comment

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

Functionality is good on the Segment debugger but coverage is down a bit

@tobitech tobitech requested a review from singhhari April 13, 2021 01:25
@@ -642,6 +642,16 @@ final class AppDelegateViewModelTests: TestCase {
XCTAssertTrue(result)

self.presentViewController.assertValues([1, 2, 2, 2, 3])

let faqUrl = projectUrl + "/faqs"
Copy link
Contributor

Choose a reason for hiding this comment

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

Are these tests unrelated to the ticket?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes it's unrelated.

[nil]
)
}
private func assertProjectRefTagEqual(
Copy link
Contributor

Choose a reason for hiding this comment

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

Can this be replaced with a standard assertion like we use everywhere else?

Copy link
Contributor Author

@tobitech tobitech Apr 13, 2021

Choose a reason for hiding this comment

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

XCTAssertEqual doesn't work with tuples because they don't conform to Equatable protocol that's why I had to do that. Another option would be to pass the Project and RefTag as separate parameters to the config() method of ProjectDescriptionViewController

@tobitech tobitech requested a review from singhhari April 13, 2021 18:30
Copy link
Contributor

@singhhari singhhari left a comment

Choose a reason for hiding this comment

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

lgtm

@tobitech tobitech merged commit 1f18c6d into master Apr 13, 2021
@tobitech tobitech deleted the EP-458-qa-page-viewed-project-phase1 branch April 13, 2021 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants