Граф коммитов

196 Коммитов

Автор SHA1 Сообщение Дата
Armen Zambrano G e8b726cd44 WIP single data loading 2018-06-08 15:18:40 -04:00
Armen Zambrano d6b4df9770
Merge pull request #178 from mozilla/sorted
Issue #57 - Allow sorting of changesets by coverage score & other fixes
2018-06-08 14:39:40 -04:00
Armen Zambrano G 1efbcf3307 Remove arrayToMap 2018-06-08 14:33:05 -04:00
Armen Zambrano G e88ba1dc55 refactor: Clean up some code 2018-06-08 14:33:05 -04:00
Armen Zambrano G 4a0f37b923 Fix filtering 2018-06-08 14:33:05 -04:00
Armen Zambrano G e3accee95c Only show icon to filter changesets when there's data 2018-06-08 14:33:05 -04:00
Armen Zambrano G de02eb6692 Issue #181 - Fix status and error messages on SummaryContainer 2018-06-08 14:33:05 -04:00
Armen Zambrano G b2aa71b744 refactor: Move coverage mocks into its own module 2018-06-08 14:33:05 -04:00
Armen Zambrano G c5e0261d46 Test sorting by coverage score 2018-06-08 14:33:05 -04:00
Armen Zambrano G 627a52f18a Test sorting of changesets by recency 2018-06-08 14:33:05 -04:00
Armen Zambrano G 5c2e9700e2 Issue #57 - Allow sorting changesets per coverage score
Add mdi-react dependency.
2018-06-08 14:31:35 -04:00
Armen Zambrano G d155896245 Initial support for sorting changesets on the main view
For now, this only supports sorting by the recency of the changeset,
however, it has the architecture to enable further sorting methods
(e.g. by coverage score).

Depending on how the data of the changeset is obtained we will use
one of two sorting set of properties (pushId/changesetIndex versus
push timestamp).
2018-06-07 11:25:37 -04:00
Armen Zambrano G 4eb3a5da6d Add daysAgoDate helper function 2018-06-07 11:11:40 -04:00
Armen Zambrano G d5bdf4ff1c refactor: Move routing logic into Routes component 2018-06-07 11:10:16 -04:00
Armen Zambrano G 2695903295 Issue #135 - Handle better the diff viewer not having coverage data
Sometimes, we would need to visit the diff viewer for old changesets.
Unfortunately, the backend would not have data available and the viewer
would fail ungracefully.

Before this, the console would be full of errors even if the UI
would not fail dramatically.
2018-06-06 09:14:20 -04:00
Armen Zambrano 12311fd4a6
Merge pull request #190 from mozilla/error_boundary
Better handle errors and fix some issues
2018-06-05 14:16:08 -04:00
Armen Zambrano G 97df61bd70 refactor: Drop BACKEND entry from settings 2018-06-05 13:51:48 -04:00
Armen Zambrano G 3d44b59beb Only enable Raven if we're in production
There's no need to try to report to sentry.io if we're doing local development.
2018-06-05 13:51:48 -04:00
Armen Zambrano G d6ac66049a Add default error boundary 2018-06-05 13:51:48 -04:00
Armen Zambrano G 464618f755 Issue #188 - TypeError: coverageMap[cset.node] is undefined 2018-06-05 10:13:50 -04:00
Armen Zambrano G 7e0091edf6 Fix regression from 2ef8399d9d 2018-06-05 10:13:46 -04:00
Armen Zambrano G 2ef8399d9d Handle better if the backend is down 2018-06-05 09:18:40 -04:00
Armen Zambrano G 56b7208260 Issue #56 - Allow filtering by changeset's description
This is useful if you want to see changesets associated to the
same bug.
2018-05-25 11:40:37 -04:00
Armen Zambrano G 4a676c668f Issue #69 - Allow folding & unfolding of files in the diff view 2018-05-24 12:32:30 -04:00
Chiranjeevi 8a150c410c Issue #66 - Added coverage color legends 2018-05-23 15:31:50 -04:00
jhkren 094c910063 Issue #132 - Added tests for coverage module 2018-05-23 15:02:11 -04:00
Armen Zambrano G 9bb0b980c6 Run tests on Travis 2018-05-22 16:49:38 -04:00
Armen Zambrano G f1696c00b7 Issue 45 - Do not show unsupported files in the Diff view 2018-05-21 08:39:39 -04:00
Armen Zambrano G e8434f8108 refactor: Create few fetching functions 2018-05-16 15:20:20 -04:00
Armen Zambrano G e479e45f08 Calling getChangesetMeta() does not have author's email and name parsed
We were not initalizing a changeset's metadata the same way as when fetched
as part of a call to jsonPushes.
2018-05-15 08:40:38 -04:00
Marco dd54aa246b
Merge pull request #171 from marco-c/update_urls
Update codecov.io and shipit URLs to point to production instances
2018-05-03 14:49:37 +02:00
Marco Castelluccio e906394e8d Use new coverage.moz.tools domain 2018-05-03 11:59:11 +02:00
Kyle Lahnakoski da9fb7bcfc
Merge pull request #168 from jankeromnes/icon-substitute-fix
I did not test this.  I am merging it anyway because the code change is simple enough to understand, and the possible negative side effects are limited to styling.
2018-04-30 09:59:59 -04:00
Kyle Lahnakoski aab431c96c
Merge pull request #169 from jankeromnes/github-css-ribbon
Good job!  Thank you!
2018-04-30 09:27:51 -04:00
Marco Castelluccio b79ebe51a1 Update codecov.io and shipit URLs to point to production instances 2018-04-27 12:40:03 +02:00
Kyle Lahnakoski d2be4802e8
Merge pull request #167 from jankeromnes/white-space-pre
Correctly display white-space in bug descriptions.
2018-04-26 21:42:02 -04:00
Jan Keromnes 3c6fb62024 Replace the pixelated GitHub ribbon with a nicer CSS version. 2018-04-19 18:19:07 +02:00
Jan Keromnes 8b1a34f1f5 Fix author placeholder icon margin. 2018-04-19 15:22:07 +02:00
Jan Keromnes ba5fe655c4 Correctly display white-space in bug descriptions. 2018-04-19 14:03:28 +02:00
Armen Zambrano G ded19f9c72 Add back support for coverage data caching
In 18cc0ed2dc
we separated the coverage data from the changesets object.

This caused the coverage data not being cached.
This commit adds support for caching the coverage data in a generic way.
2018-03-29 10:31:38 -04:00
Armen Zambrano G caae29517e Issue #159 - Fix: Regression from 18cc0ed2dc
In the past we used to determine if a changeset was to show by doing this check:

```javascript
mapToArray(changesets).filter(c => c.hidden === false)
```

We then switched to:

```javascript
coverageMap[cset.node].summary !== PENDING
```

This adds a `show` property and we check on it:

```javascript
coverageMap[cset.node].show
```
2018-03-29 09:52:43 -04:00
Armen Zambrano G 245afefb79 Issue #138 - DiffViewer - Add link to the bug and show the commit message 2018-03-29 09:52:43 -04:00
Armen Zambrano G 736fa1ec52 Move Bugzilla icon to the left 2018-03-28 16:07:14 -04:00
Kathy Ren a0cae4ab3e Issue #72 - Lines with no coverage information are marked as miss 2018-03-28 09:52:49 -04:00
Dottori 6915ea5cee Issue #122 - SummaryViewer: Fix truncated author on main page
This change was missed during rebasing and fixing conflicts.

Without it, the description would not be ligned up to the left.
2018-03-28 09:41:16 -04:00
Armen Zambrano G 35c82253d4 Refactor: Make repoName an implicit value in the Hg module
There was a mix of `repoName` versus `repoPath` in the signature of
various signatures.

Since this is not a feature that we really support (switch between repos)
let's be consistent and not have to pass it around as a property.
2018-03-28 09:33:42 -04:00
Armen Zambrano G 557fa2fd89 Refactor: Split out diffViewer modules 2018-03-28 09:33:42 -04:00
Armen Zambrano G 18cc0ed2dc Refactor: Separate coverage data into its own entity
This will simplify getting coverage data needed on different views.

For instance, we should not need to fetch changeset data when
we already have the changeset's node available and we can directly
fetch the coverage data.

In a future pass we will now be able to store in the local cache the coverage data
separate from the changesets data.

We also drop the notion of hidden changesets since we were not
taking advantage of it.
2018-03-28 09:33:42 -04:00
Armen Zambrano G 21a1d365a8 Refactor: Move coverage code into a new coverage module 2018-03-28 09:33:42 -04:00
Armen Zambrano G 96e709ed43 Refactor: Move Hg raw diff code into the hg module 2018-03-28 09:33:42 -04:00