Skip to content

✅ add test for phone Input component#2554

Merged
celo-ci-bot-user merged 4 commits intomasterfrom
aaronmgdr/phoneInputSpec
Jan 28, 2020
Merged

✅ add test for phone Input component#2554
celo-ci-bot-user merged 4 commits intomasterfrom
aaronmgdr/phoneInputSpec

Conversation

@aaronmgdr
Copy link
Copy Markdown
Contributor

Description

adds tests for PhoneInput Component

Tested

yep

Other changes

add a custom onPress test event trigger

Testing interactivity and interplay between the two input elements
@aaronmgdr aaronmgdr requested a review from cmcewen as a code owner January 27, 2020 21:10
@jmrossy jmrossy assigned aaronmgdr and unassigned jmrossy Jan 27, 2020
@jmrossy
Copy link
Copy Markdown
Contributor

jmrossy commented Jan 27, 2020

Lint failures

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 27, 2020

Codecov Report

Merging #2554 into master will increase coverage by 0.01%.
The diff coverage is 87.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2554      +/-   ##
==========================================
+ Coverage    73.9%   73.91%   +0.01%     
==========================================
  Files         556      557       +1     
  Lines       13820    13826       +6     
  Branches     1425     1425              
==========================================
+ Hits        10213    10219       +6     
  Misses       3326     3326              
  Partials      281      281
Flag Coverage Δ
#mobile 74.1% <ø> (ø) ⬆️
#web 73.63% <87.5%> (+0.02%) ⬆️
Impacted Files Coverage Δ
packages/web/src/header/Header.3.tsx 0% <0%> (ø) ⬆️
packages/web/src/shared/Footer.tsx 87.8% <100%> (ø)
packages/web/src/icons/YouTube.tsx 100% <100%> (ø)
packages/web/src/shared/menu-items.ts 100% <100%> (ø) ⬆️
packages/web/src/brandkit/common/Page.tsx 63.51% <100%> (ø) ⬆️

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 897e219...82c9a35. Read the comment docs.

@jmrossy jmrossy removed their assignment Jan 28, 2020
@aaronmgdr
Copy link
Copy Markdown
Contributor Author

gracias

@celo-ci-bot-user celo-ci-bot-user merged commit ddb25e0 into master Jan 28, 2020
@celo-ci-bot-user celo-ci-bot-user deleted the aaronmgdr/phoneInputSpec branch January 28, 2020 02:16
aaronmgdr added a commit that referenced this pull request Jan 28, 2020
* master:
  🧹Web cleanup (readme + static dir) (#2562)
  Add readable proposals to governance:view command (#2545)
  Add explicit gas to exchange transactions to prevent errors (#2552)
  Fix off-by-one error in attributing signatures to blocks in CLI (#2559)
  ✅ add test for phone Input component (#2554)
  Add Youtube to Footer +  (#2556)
  Fix rounding error in Election.sol (#2540)
  [Wallet] Bump @celo/client to 0.0.266 (#2551)
  [Wallet] E2E test improvements (#2542)
  Deployed integration (#2550)
  do not fetch affiliates (#2508)
  Added more CLI checks for registering validators and groups (#2491)
  Micro Improvement to web tests (#2527)
  [Wallet] Prompt users with connectivity issues to switch to forno (#2526)
  cli: Fix voter rewards presentation (#2543)
  [Wallet] Fix missing spanish translation  (#2539)
  Downtime slashing when epoch changes (#2436)
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.

3 participants