Armen Zambrano G
06d09b094b
Refactor: Clean up local caching code
2018-03-28 09:33:42 -04:00
Armen Zambrano G
71a5aa8aa0
Refactor: Split up some modules + rename as .jsx files
2018-03-28 09:33:42 -04:00
Dottori
3a4ce2c9e7
Issue #122 - SummaryViewer: Fix truncated author on main page
2018-03-27 16:30:11 -04:00
Armen Zambrano G
3a219660d3
Fix: Lint src/ again instead of just test/
2018-03-26 11:50:31 -04:00
Armen Zambrano G
c3fc6fb642
Major refactor: Move all Hg logic into its own module
...
This decouples all Mercurial logic from ChangesetsViewerContainer.
This enables other containers to fetch data from hg.mozilla.org
Only enable the cache when running on production.
Allow enabling the cache when doing local development with ENABLE_CACHE=true
2018-03-23 11:38:22 -04:00
Armen Zambrano G
b02ee8360d
Refactor settings into a single object
...
Also move more constants into it.
2018-03-23 09:00:03 -04:00
Armen Zambrano G
a0d0ed6cb9
Add /clear-chache route to clear local cache
2018-03-23 09:00:03 -04:00
Armen Zambrano G
2ff1928e81
Fetch pushes from 3 days ago
2018-03-23 09:00:03 -04:00
Armen Zambrano G
64a9a9f8f7
Issue #103 - If the backend has no coverage data clear the 'Loading...' message
2018-03-23 09:00:03 -04:00
Kathy Ren
95a1db6e8e
Issue #132 - Add initial test
...
Add @neutrinojs/mocha as a dependency.
We're testing a couple of methods from utils/data.js
Configure eslint to lint the 'test/' directory.
2018-03-23 08:57:53 -04:00
Armen Zambrano G
62aa5bf17f
Issue #153 - Fix: The DiffViewer should not fail if the coverage data for a changeset is PENDING
...
Normally, from the SummaryViewer you should not be able to get to the DiffViewer since
there's no link to navigate to it.
However, people clicking on links from GitHub issues can reach the DiffViewer, thus,
failing because there `coverage` object contains a summary property rather than
being undefined.
This change shows an error message to the user letting them know that they should
try again later.
There's also some linting fixes in this change.
2018-03-20 11:56:17 -04:00
Armen Zambrano G
6ff2b3d96a
Reduce build size. Only import the functions we need from lodash
2018-03-20 11:56:17 -04:00
Armen Zambrano G
504e5baff3
Remove some custom linting rules + fixes for them
2018-03-20 11:56:17 -04:00
jhkren
09cb429bc6
Issue #115 : FileViewer: Accept any length revision number ( #146 )
...
Query ActiveData by prefix matching revision number id. Add minimum revision number length check before AD query. Show error message if revision is too short.
2018-03-19 10:41:45 -04:00
Armen Zambrano G
cb40609d18
Issue #150 - fix: GitHub ribbon should only show up on the summary view
2018-03-14 14:35:25 -04:00
Chiranjeevi SD
667ac29fd0
Issue #136 - Add GitHub ribbon
2018-03-14 10:25:51 -04:00
Armen Zambrano G
135da3f7ae
Issue #143 - Show network error message when failing to reach Hg
...
Failure to fetch raw diff should show a network error message insted of a parsing issue.
2018-03-12 10:12:23 -04:00
Armen Zambrano G
36f362f726
Issue #141 - Generate source maps for production
2018-03-08 15:24:07 -05:00
Armen Zambrano G
619f47da78
Issue #139 - Use sentry.io to track frontend runtime issues
2018-03-08 12:19:39 -05:00
Armen Zambrano G
e0a68277ec
Issue #126 - Upgrade to parse-diff 0.4.2
2018-03-05 12:43:43 -05:00
Dottori
abf360542c
Issue #119 - Cache changesets for SummaryViewer ( #129 )
...
This drastically improves the usability of the app by reducing the time it takes for the main page to load on subsequent loads.
Implemented behaviour:
If the changesets have not been cached fetch them.
If the cache is older than 24 hours, clear it and fetch new changesets.
2018-03-01 08:15:11 -05:00
Armen Zambrano G
45750226c9
Prevent commiting code without fixing linting issues first
2018-02-23 13:53:10 -05:00
jhkren
b52dbe2c07
Issue #107 - Changeset in main page to link to Hg and coverage summary to navigate to the diff viewer ( #121 )
2018-02-23 09:24:02 -05:00
Dottori
b1db97ca9f
Issue #99 : SummaryViewer: modify appearance of changeset description ( #118 )
2018-02-20 14:50:00 -05:00
jhkren
2c4717f534
Issue #120 : Fix - Bad coverage calculation
...
Now checks for `fileDiffs.lines` for coverage status instead of `fileDiffs`.
2018-02-20 13:57:38 -05:00
Dottori
ddd9942b0c
Issue #79 - Move coverage percentage logic to coverage parse logic ( #124 )
2018-02-15 15:19:29 -05:00
Dottori
26e585893a
Issue #85 - Add link to fileviewer in diffblocks to view a file's coverage
2018-02-13 12:35:12 -05:00
Abhay Chawla
98329ad3e5
Issue #21 - Show 'Loading...' message while coverage data is not yet available ( #112 )
2018-02-09 12:20:53 -05:00
jhkren
c532cffd21
Issue #67 - DiffViewer - Sort files by least coverage at the top ( #100 )
2018-02-09 11:13:53 -05:00
Armen Zambrano
2ff3050eab
Issue #98 - Switch from react-scripts to Neutrino ( #109 )
...
Add NPM_CONFIG_PRODUCTION=false in order for Neutrino to get installed
before heroku-postbuild is run.
2018-02-09 08:37:49 -05:00
Armen Zambrano G
1e53e125a7
Issue #110 - 'Known issues' should point to the upstream repo
2018-02-08 11:27:26 -05:00
jhkren
b9e35dacc4
Issue #71 - Handle renamed and added files properly
2018-02-07 12:53:56 -05:00
Dottori
845c1cb5d0
Issue #97 - remove calls to diffMeta and parentMeta ( #105 )
2018-02-06 13:59:41 -05:00
Dottori
df5a39a4f3
Issue #75 - Reshuffled some links for the DiffViewer ( #93 )
2018-01-31 07:59:14 -05:00
Dottori
06927afa3a
Issue 90 - Add bugzilla links to the summary viewer ( #88 )
2018-01-29 16:35:26 -05:00
Armen Zambrano
f3a6b228cd
docs - Add note on how to keep branch up-to-date
2018-01-29 10:26:19 -05:00
Armen Zambrano G
70434d4cc2
Issue 26 - Travis support with linting check
2018-01-29 10:06:44 -05:00
Armen Zambrano
2f572337db
docs - Guide on how to contribute to the project ( #94 )
...
This guide helps new contributors know what to expect when collaborating.
2018-01-22 16:27:09 -05:00
Armen Zambrano
d830ab7c56
docs - Link to overall architecture of Mozilla's code coverage blog post ( #95 )
...
Plus fixing minor issues.
2018-01-22 16:21:51 -05:00
Armen Zambrano G
671366674d
File coverage viewer
...
This view allows you to select each line of a file
and determine which tests cover that specific line.
This work comes from the UCOSP work from @LinkaiQi and @yuenj.
2018-01-03 09:02:01 -05:00
Armen Zambrano G
700e78b2b4
Upgrade react-scripts to fix a mime low threat vulnerability
...
This fixes one or more vulnerable packages in the npm dependencies of this project. See the [Snyk test report](https://snyk.io/test/github/armenzg/firefox-code-coverage-frontend ) for this project for details.
This change includes:
- Changes to `package.json` and `yarn.lock` to upgrade the vulnerable dependencies to a fixed version.
- [npm:mime:20170907](https://snyk.io/vuln/npm:mime:20170907 )
2017-11-13 08:44:23 -05:00
Armen Zambrano G
03c1c5a3b7
refactor: Transform coverage data to improve internal manipulation
2017-11-10 14:36:19 -05:00
Armen Zambrano G
2100556e12
Issue #61 - fix: DiffViewer fails in certain changesets
...
Under certain circumstances the coverage data can be
incorrect and we need to safe guard against it.
2017-11-10 14:36:19 -05:00
Armen Zambrano G
0a13f582df
Issue #44 - docs: Update app's disclaimers and documentation
2017-11-10 10:04:55 -05:00
Armen Zambrano G
e8e5b5aad4
refactor: DiffViewer to use same common code to generate code coverage status
2017-11-08 15:43:59 -05:00
Armen Zambrano G
83b50612ad
refactor: Create settings module
2017-11-08 15:43:59 -05:00
Armen Zambrano G
6d746e4b1c
feat: Show coverage status summary for each changeset
2017-11-08 15:43:59 -05:00
Armen Zambrano G
5d27536c90
refactor: Move data structure functions into its own module
2017-11-08 15:43:59 -05:00
Armen Zambrano G
1cc809b2c9
refactor: Move fetch_data module under src/utils/
2017-11-08 15:43:59 -05:00
Armen Zambrano G
9c8b585766
refactor: Do not show table headers while loading
2017-11-08 15:43:59 -05:00