Commit graph

7915 commits

Author SHA1 Message Date
Paul Chavard
d8ba8d72e0
Merge pull request #3915 from betagouv/disable-map-tiles-during-tests
carte: disable tiles layer during tests
2019-06-04 11:00:53 +02:00
Pierre de La Morinerie
f2f16e2580 carte: disable tiles layer during tests
During integration tests, we don't want to load the tiles from OSM:

- It hits OSM servers during every test run;
- It it slow (Capybara waits for the tiles to be loaded to proceed);
- It makes test time out when tiles cannot be loaded for some reason.

Fix #3913
2019-06-03 17:24:15 +02:00
Pierre de La Morinerie
c7cde75b53
Merge pull request #3912 from betagouv/improve-specs
Amélioration de la fiabilité des tests d'acceptance
2019-06-03 14:07:44 +02:00
Pierre de La Morinerie
a555b24675 specs: precompile Webpack assets before running the specs suite
This avoids the first feature spec stalling for a few dozens of seconds
because Webpack assets are compiling (and thus reduce the risk of the
spec timing out).

Pre-compilation takes ~ 10s.
2019-06-03 12:50:59 +02:00
Pierre de La Morinerie
2bd749a325 specs: clear the React champ before adding a new value
This prevents the value from being
`Nouveau champ TexteLibellé de champ carte`.
2019-06-03 12:47:03 +02:00
Pierre de La Morinerie
c05dc38b95 specs: remove 'disable-gpu' from chromedriver options
This option as only ever needed on Windows [1] (which we don't use),
and it shouldn't be required anymore, as the underlying bug has been
fixed [2].

- [1] https://developers.google.com/web/updates/2017/04/headless-chrome
- [2] https://bugs.chromium.org/p/chromium/issues/detail?id=737678
2019-06-03 12:46:12 +02:00
Pierre de La Morinerie
4feecb0833
Bump fstream from 1.0.11 to 1.0.12 (#3910)
- Bump de la dépendance indirecte `fstream`
2019-06-03 10:56:26 +02:00
dependabot[bot]
6393401f37
Bump fstream from 1.0.11 to 1.0.12
Bumps [fstream](https://github.com/npm/fstream) from 1.0.11 to 1.0.12.
- [Release notes](https://github.com/npm/fstream/releases)
- [Commits](https://github.com/npm/fstream/compare/v1.0.11...v1.0.12)
2019-05-31 10:15:16 +00:00
Paul Chavard
db2b531656
Merge pull request #3903 from tchak/purge-pj
Ajouter la possibilité de supprimer une pj sur les avis
2019-05-29 16:03:46 +02:00
Paul Chavard
ff44b7a600 Refactor purge pj to be more generic 2019-05-29 15:54:51 +02:00
Pierre de La Morinerie
4013557cd7
Améliorations de robustesse à la tâche de migration des PR (#3904)
Améliorations de robustesse à la tâche de migration des PR
2019-05-29 14:56:04 +02:00
Pierre de La Morinerie
a8847e40ea pj migration: add comments and notices 2019-05-29 14:46:49 +02:00
Pierre de La Morinerie
6cb02f2927 pj migration: handle signal interrupts 2019-05-29 14:46:49 +02:00
Pierre de La Morinerie
1a256b37f8 pj migration: print a notice when migrating missing files 2019-05-29 12:20:10 +02:00
Pierre de La Morinerie
a168ae3a72 pj migration: reduce the number of queries by preloading champs 2019-05-29 12:20:10 +02:00
Pierre de La Morinerie
60121a1be6 task_helper: ensure that incrementing above 100% doesn’t raise an error 2019-05-29 12:06:32 +02:00
Pierre de La Morinerie
2bb103b8bd task_helper: mark private methods 2019-05-29 11:48:41 +02:00
Pierre de La Morinerie
4d9cb0bb7a task_helper: fix duration formatting
If we don't convert the duration to utc, it starts at 01:00:00
2019-05-29 11:48:21 +02:00
Pierre de La Morinerie
f1875e9c25
Corrections sur la tâche de migration des PJs (#3902)
Corrections sur la tâche de migration des pièces justificatives
2019-05-29 11:07:54 +02:00
Pierre de La Morinerie
10df7b70ee carrierwave: when migrating, create an empty blob if file is missing 2019-05-28 18:19:33 +02:00
Pierre de La Morinerie
44c410d40d piece_justificative_service: fix for missing order_place
In production some pieces justificatives don't have an order place.

In this case, insert the champs after the ones that have an order place.
2019-05-28 17:20:07 +02:00
Pierre de La Morinerie
c7f2113972 tasks: add a description to make :pieces_justificatives tasks valid 2019-05-28 15:29:59 +02:00
Paul Chavard
a65774b726 Fix job enqueue instruction in README 2019-05-28 12:24:44 +02:00
Pierre de La Morinerie
de22f09ea6
Merge pull request #3883 from betagouv/mark-migrated-pjs-as-scanned
Migration des pièces-jointes : évite de faire tourner l'antivirus lors de la migration
2019-05-28 11:58:37 +02:00
Pierre de La Morinerie
d410e31344 active_storage: document the virus scan hooks 2019-05-28 11:39:22 +02:00
Pierre de La Morinerie
52b7a82932 services: mark attachments migrated from CarrierWave as safe
This avoids to enqueue thousands of scans when migrating the PJs of
a whole procedure.
2019-05-28 11:39:22 +02:00
Paul Chavard
01e6af47a7
Merge pull request #3880 from betagouv/batch-migrate-pjs
Ajout d'une tâche pour migrer les pièces justificatives de plusieurs démarches
2019-05-28 11:29:53 +02:00
Pierre de La Morinerie
4cf54e0d28 tasks: add progress report to the pjs migration task
Progress is indicated per migrated champ.
2019-05-28 10:42:18 +02:00
Pierre de La Morinerie
21ac60ad04 tasks: add a task to migrate pjs of procedures in batches 2019-05-28 10:42:18 +02:00
Paul Chavard
0f59f0fd4a
Merge pull request #3898 from betagouv/fix-3896-tracking
Add Gestionnaire ID and roles in SB tracker
2019-05-28 09:14:02 +02:00
Chaïb Martinez
920c8c0c32 Add Gestionnaire ID and ROLES
Signed-off-by: Chaïb Martinez <chaibax@gmail.com>
2019-05-27 16:30:03 +02:00
Keirua
ce5ee0b766
Merge pull request #3897 from Keirua/fix/crash-empty-query-search
no crash when q is missing on RechercheController::index
2019-05-22 15:03:31 +02:00
clemkeirua
f06ae1631f no crash when q is missing on RechercheController::index 2019-05-22 15:02:57 +02:00
Paul Chavard
64009a5e10
Merge pull request #3892 from tchak/refresh-virus-scanner-checks
Refresh attachments with virus scan result
2019-05-22 14:46:02 +02:00
Paul Chavard
6797c01b69 Properly handle justificatif_motivation case 2019-05-22 14:11:08 +02:00
Paul Chavard
6a3413018a Refresh attachments with virus scan result 2019-05-21 14:21:55 +02:00
Paul Chavard
cc4eba2b36 Less mokey patching 2019-05-21 14:21:42 +02:00
Paul Chavard
f6421e081a Refactor attachment view 2019-05-21 14:21:13 +02:00
Paul Chavard
b55fae440b
Merge pull request #3891 from tchak/purge-orphaned-blobs
Purge unattached blobs
2019-05-21 14:18:14 +02:00
Paul Chavard
f833f57e45 Add PurgeUnattachedBlobsJob 2019-05-21 14:05:33 +02:00
Paul Chavard
4a9ef5d12e Always use purge_later 2019-05-21 14:05:33 +02:00
LeSim
3a5a051018
Merge pull request #3861 from betagouv/fix-3856-mailto
Fix 3856 mailto
2019-05-20 15:53:55 +02:00
Chaïb Martinez
49f0127300 [fix #3856] Add form contact admin link
Signed-off-by: Chaïb Martinez <chaibax@gmail.com>
2019-05-20 15:30:28 +02:00
LeSim
ca0e5aa004
Merge pull request #3873 from betagouv/fix-3867-texte-juridique
Fix 3867 texte juridique
2019-05-20 13:56:15 +02:00
Chaïb Martinez
855d4d0245 [fix #3867] Add legal text
Signed-off-by: Chaïb Martinez <chaibax@gmail.com>
2019-05-20 13:43:46 +02:00
LeSim
eeea6c3d79
Merge pull request #3885 from betagouv/fix-unclosed-tags
Fix unclosed div tags in manager views
2019-05-20 13:36:39 +02:00
Nicolas Bouilleaud
e423da9319 Fix unclosed div tags in manager views 2019-05-20 13:30:52 +02:00
Pierre de La Morinerie
5fdb8eb2de
Fix irb_context in rails console (#3889)
Suppression d'un message d'erreur qui apparaissait au démarrage de la console Rails
2019-05-20 10:46:37 +02:00
Nicolas Bouilleaud
76925356b6 Fix irb_context in rails console
Don’t require rspec_junit_formatter gem
It’s only needed when running tests (for circleci).
It messes with the console, when running `rails c` it causes this warning:
```
irb: warn: can't alias context from irb_context
```
This is related to rspec monkey_patching a `context` method on `Object` (rspec/rspec-rails#1645)
2019-05-20 10:33:43 +02:00
Keirua
f69fa4142f
Merge pull request #3887 from Keirua/fix/more-domains-csp
CSP: more whitelist for the common domains we use
2019-05-20 10:18:50 +02:00