1363130 - remove integration test

Status: RESOLVED FIXED

(Socorro :: General, task)

Description

[Will Kahn-Greene [:willkg] ET needinfo? me]

Socorro has an integration test (scripts/integration-test.sh) that runs as part of Travis tests.

The integration test currently tests the interaction between the Socorro collector/crashmover and the processor. Since we switched from using the Socorro collector/crashmover to Antenna, the integration test isn't testing anything helpful for us anymore.

We have two options:

1. update the integration test to use Antenna
2. remove it altogether

We're in the process of extracting and rewriting the processor, too, so I'm voting we remove the integration test altogether.

This bug covers removing the code, any documentation, and anywhere it's used.

Comment 1

[Will Kahn-Greene [:willkg] ET needinfo? me]

Grabbing this to do now. This will speed up the tests that run for every PR a bit.

Comment 2

[Will Kahn-Greene [:willkg] ET needinfo? me]

Attachment: github PR

Comment 3

[[github robot]]

Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/6ffefdce8a1ace32b39d88d756d3ca84f8f5b244
1363130 integration test (#3759)

* Fixes bug 1363130 - remove integration test

* Fixes bug 1363130 - remove Collector2015App, BreakpadCollector2015

The Socorro collector had a 2015 version of a few classes that had a different
url registration and allowed for collecting of other non-Breakpad things. The
classes were used in the integration tests and also had unit tests.

We (Mozilla) never ended up switching to that, so this removes all of it because
we're not going to use it and it's no longer used in the integration tests.