Skip to content

Commit 80bf05c

Browse files
committed
Fix config closure and added tests
1 parent f1ad986 commit 80bf05c

File tree

3 files changed

+62
-7
lines changed

3 files changed

+62
-7
lines changed

src/Config.php

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -31,13 +31,7 @@ public static function encryptEmails()
3131
*/
3232
public static function testing()
3333
{
34-
$testing = config('laravel-database-emails.testing.enabled', function () {
35-
return function () {
36-
return false;
37-
};
38-
});
39-
40-
return $testing();
34+
return (boolean) config('laravel-database-emails.testing.enabled', false);
4135
}
4236

4337
/**

tests/ConfigTest.php

Lines changed: 59 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,59 @@
1+
<?php
2+
3+
namespace Tests;
4+
5+
6+
use Stackkit\LaravelDatabaseEmails\Config;
7+
8+
class ConfigTest extends TestCase
9+
{
10+
/** @test */
11+
public function test_max_attempt_count()
12+
{
13+
$this->assertEquals(3, Config::maxAttemptCount());
14+
15+
$this->app['config']->set('laravel-database-emails.attempts', 5);
16+
17+
$this->assertEquals(5, Config::maxAttemptCount());
18+
}
19+
20+
/** @test */
21+
public function test_encrypt_emails()
22+
{
23+
$this->assertFalse(Config::encryptEmails());
24+
25+
$this->app['config']->set('laravel-database-emails.encrypt', true);
26+
27+
$this->assertTrue(Config::encryptEmails());
28+
}
29+
30+
/** @test */
31+
public function test_testing()
32+
{
33+
$this->assertFalse(Config::testing());
34+
35+
$this->app['config']->set('laravel-database-emails.testing.enabled', true);
36+
37+
$this->assertTrue(Config::testing());
38+
}
39+
40+
/** @test */
41+
public function test_test_email_address()
42+
{
43+
$this->assertEquals('[email protected]', Config::testEmailAddress());
44+
45+
$this->app['config']->set('laravel-database-emails.testing.email', '[email protected]');
46+
47+
$this->assertEquals('[email protected]', Config::testEmailAddress());
48+
}
49+
50+
/** @test */
51+
public function test_cronjob_email_limit()
52+
{
53+
$this->assertEquals(20, Config::cronjobEmailLimit());
54+
55+
$this->app['config']->set('laravel-database-emails.limit', 15);
56+
57+
$this->assertEquals(15, Config::cronjobEmailLimit());
58+
}
59+
}

tests/TestCase.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,8 @@ protected function getPackageProviders($app)
7979
protected function getEnvironmentSetUp($app)
8080
{
8181
$app['config']->set('laravel-database-emails.attempts', 3);
82+
$app['config']->set('laravel-database-emails.testing.enabled', false);
83+
$app['config']->set('laravel-database-emails.testing.email', '[email protected]');
8284

8385
$app['config']->set('database.default', 'testbench');
8486
$app['config']->set('database.connections.testbench', [

0 commit comments

Comments
 (0)