diff --git a/tests/TestCase.php b/tests/TestCase.php
index 967575c5fb..f809d58ace 100644
--- a/tests/TestCase.php
+++ b/tests/TestCase.php
@@ -99,7 +99,13 @@ abstract class TestCase extends Illuminate\Foundation\Testing\TestCase
*/
public function naughtyStringProvider()
{
+ /*
+ * If on Travis, return very small set.
+ */
+ if (getenv('TRAVIS') == '1') {
+ return [['Default value']];
+ }
$path = realpath(__DIR__ . '/../resources/tests/blns.base64.json');
$content = file_get_contents($path);
$array = json_decode($content);
diff --git a/tests/acceptance/Controllers/Transaction/SingleControllerTest.php b/tests/acceptance/Controllers/Transaction/SingleControllerTest.php
index f186514ecc..81e67a4e21 100644
--- a/tests/acceptance/Controllers/Transaction/SingleControllerTest.php
+++ b/tests/acceptance/Controllers/Transaction/SingleControllerTest.php
@@ -106,29 +106,6 @@ class SingleControllerTest extends TestCase
$this->assertSessionHas('success');
}
- /**
- * @covers \FireflyIII\Http\Controllers\Transaction\SingleController::store
- * @dataProvider naughtyStringProvider
- */
- public function testStoreNaughty(string $description)
- {
- $this->session(['transactions.create.url' => 'http://localhost']);
- $this->be($this->user());
-
- $data = [
- 'what' => 'withdrawal',
- 'amount' => '10',
- 'amount_currency_id_amount' => 1,
- 'source_account_id' => 1,
- 'destination_account_name' => 'Some destination',
- 'date' => '2016-01-01',
- 'description' => $description,
- ];
- $response = $this->call('post', route('transactions.store', ['withdrawal']), $data);
- $this->assertNotEquals($response->getStatusCode(), 500);
-
- }
-
/**
* @covers \FireflyIII\Http\Controllers\Transaction\SingleController::update
*/
@@ -161,5 +138,4 @@ class SingleControllerTest extends TestCase
$this->see('
');
}
-
}
diff --git a/tests/acceptance/NaughtyListTest.php b/tests/acceptance/NaughtyListTest.php
new file mode 100644
index 0000000000..d729ff362e
--- /dev/null
+++ b/tests/acceptance/NaughtyListTest.php
@@ -0,0 +1,123 @@
+where('user_id', $this->user()->id)
+ ->whereNull('deleted_at')
+ ->first();
+ $this->session(['transactions.mass-edit.url' => 'http://localhost']);
+
+ $data = [
+ 'journals' => [$deposit->id],
+ 'description' => [$deposit->id => $description],
+ 'amount' => [$deposit->id => 1600],
+ 'amount_currency_id_amount_' . $deposit->id => 1,
+ 'date' => [$deposit->id => '2014-07-24'],
+ 'source_account_name' => [$deposit->id => 'Job'],
+ 'destination_account_id' => [$deposit->id => 1],
+ 'category' => [$deposit->id => 'Salary'],
+ ];
+
+ $this->be($this->user());
+ $response = $this->call('post', route('transactions.mass.update', [$deposit->id]), $data);
+ $this->assertNotEquals($response->getStatusCode(), 500);
+
+ // visit them should show updated content
+ $this->call('get', route('transactions.show', [$deposit->id]));
+ $this->assertResponseOk();
+ $this->see($description);
+ }
+
+ /**
+ * @covers \FireflyIII\Http\Controllers\Transaction\SingleController::store
+ * @dataProvider naughtyStringProvider
+ *
+ * @param string $description
+ */
+ public function testSingleStore(string $description)
+ {
+ $this->session(['transactions.create.url' => 'http://localhost']);
+ $this->be($this->user());
+
+ $data = [
+ 'what' => 'withdrawal',
+ 'amount' => '10',
+ 'amount_currency_id_amount' => 1,
+ 'source_account_id' => 1,
+ 'destination_account_name' => 'Some destination',
+ 'date' => '2016-01-01',
+ 'description' => $description,
+ ];
+ $response = $this->call('post', route('transactions.store', ['withdrawal']), $data);
+ $this->assertNotEquals($response->getStatusCode(), 500);
+
+ }
+
+ /**
+ * @covers \FireflyIII\Http\Controllers\Transaction\SingleController::update
+ * @dataProvider naughtyStringProvider
+ *
+ * @param string $description
+ */
+ public function testSingleUpdate(string $description)
+ {
+ $this->session(['transactions.edit.url' => 'http://localhost']);
+ $this->be($this->user());
+ $data = [
+ 'id' => 123,
+ 'what' => 'withdrawal',
+ 'description' => $description,
+ 'source_account_id' => 1,
+ 'destination_account_name' => 'PLUS',
+ 'amount' => '123',
+ 'amount_currency_id_amount' => 1,
+ 'budget_id' => 1,
+ 'category' => 'Daily groceries',
+ 'tags' => '',
+ 'date' => '2016-01-01',
+ ];
+
+ $response = $this->call('post', route('transactions.update', [123]), $data);
+ $this->assertNotEquals($response->getStatusCode(), 500);
+ $this->assertSessionHas('success');
+
+ $this->call('get', route('transactions.show', [123]));
+ $this->assertResponseStatus(200);
+ $this->see($description);
+ // has bread crumb
+ $this->see('');
+
+ }
+
+}
\ No newline at end of file