Skip to content

Conversation

@jcalarcon98
Copy link
Contributor

Description

Currently there is an error with a test, which only fails at the end of the month. The test that fails is as follows:
image

Solution

This test has been corrected so that it does not fail again on these dates.

@jcalarcon98 jcalarcon98 requested a review from EdansRocks August 30, 2021 17:16
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jcalarcon98 jcalarcon98 changed the title fix: TT-325 Fix wront Test in calendar component fix: TT-325 Fix wrong Test in calendar component Aug 30, 2021
@codecov
Copy link

codecov bot commented Aug 30, 2021

Codecov Report

Merging #741 (3fdabc1) into master (9a68415) will increase coverage by 0.16%.
The diff coverage is 96.42%.

❗ Current head 3fdabc1 differs from pull request most recent head d21e7a9. Consider uploading reports for the commit d21e7a9 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #741      +/-   ##
==========================================
+ Coverage   95.24%   95.40%   +0.16%     
==========================================
  Files         106      106              
  Lines        2206     2262      +56     
  Branches      169      171       +2     
==========================================
+ Hits         2101     2158      +57     
  Misses         61       61              
+ Partials       44       43       -1     
Impacted Files Coverage Δ
src/app/app-routing.module.ts 100.00% <ø> (ø)
...les/shared/components/sidebar/sidebar.component.ts 90.90% <ø> (-1.69%) ⬇️
.../components/entry-fields/entry-fields.component.ts 97.18% <ø> (-0.04%) ⬇️
.../components/technologies/technologies.component.ts 85.18% <83.33%> (+0.81%) ⬆️
...ponents/details-fields/details-fields.component.ts 96.40% <100.00%> (+0.16%) ⬆️
...c/app/modules/shared/utils/project-storage.util.ts 100.00% <100.00%> (ø)
...project-list-hover/project-list-hover.component.ts 90.74% <100.00%> (+0.35%) ⬆️
...-entries/components/calendar/calendar.component.ts 95.38% <100.00%> (+0.22%) ⬆️
...dules/time-entries/pages/time-entries.component.ts 88.67% <100.00%> (+0.67%) ⬆️
...sers/components/users-list/users-list.component.ts 100.00% <100.00%> (ø)
... and 5 more

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 ffc8ed9...d21e7a9. Read the comment docs.

@jcalarcon98 jcalarcon98 merged commit cfef777 into master Aug 30, 2021
@jcalarcon98 jcalarcon98 deleted the TT-325-fix-wrong-tests-in-calendar-component branch August 30, 2021 17:26
bytesantiago added a commit that referenced this pull request Aug 30, 2021
* perf: TT-316 removal of ui-list-technologies toggle, technologies-report.guard and their dependants

* perf: TT-317 remove unnecessary features from FeatureManagerService

* code-smell: TT-316 remove comments and unnecessary variable assignments

* code-smell: TT-316 removal of an additional unnecessary import

* Tt 319 fix calendar day week view (#738)

* fix: TT-319 Fix calendar week and day view

* refactor: TT-319 select first day of the month when select on calendar

* refactor: TT-319 Change moment library with date

Co-authored-by: Andrés Soto <edansrock>

* perf: TT-316 Remove technologies-report-guard/ and remove the feature toggle  ui-list-technologies (#736)

* perf: TT-316 removal of ui-list-technologies toggle, technologies-report.guard and their dependants

* code-smell: TT-316 remove comments and unnecessary variable assignments

* code-smell: TT-316 removal of an additional unnecessary import

* chore(release): 1.50.1 [skip ci]nn

* perf: TT-317 remove unnecessary features from FeatureManagerService

* fix: TT-325 Fix wront Test in calendar component (#741)

* chore(release): 1.50.2 [skip ci]nn

* perf: TT-317 remove unnecessary features from FeatureManagerService

* code-smell: TT-317 remove code smells indicated by sonarcloud

Co-authored-by: EdansRocks <[email protected]>
Co-authored-by: semantic-release-bot <[email protected]>
Co-authored-by: Jean Carlos Alarcón <[email protected]>
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