Skip to content

Commit

Permalink
drop 7.2 & 7.3
Browse files Browse the repository at this point in the history
  • Loading branch information
nadar committed Mar 16, 2023
1 parent f1c8ac3 commit 8d4aaa3
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-latest]
php: ['7.2', '7.3', '7.4', '8.0', '8.1', '8.2']
php: ['7.4', '8.0', '8.1', '8.2']

steps:
## checkout the repoistory
Expand All @@ -77,7 +77,7 @@ jobs:
## run unit tests
- name: PHP Unit tests for PHP
run: vendor/bin/phpunit --verbose --configuration actions.phpunit.xml
if: matrix.php == '7.2' || matrix.php == '7.3' || matrix.php == '8.0' || matrix.php == '8.1' || matrix.php == '8.2'
if: matrix.php == '8.0' || matrix.php == '8.1' || matrix.php == '8.2'

## unit test with coverage
- name: PHP Unit tests for PHP 7.4 Coverage
Expand Down
10 changes: 5 additions & 5 deletions tests/controllers/DefaultControllerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ public function testEmailMessage()
EOT;

$ctrl = new DefaultController('default', Yii::$app->getModule('contactform'));
$this->assertContains($mail, $ctrl->generateMailMessage($model));
$this->assertStringContainsString($mail, $ctrl->generateMailMessage($model));
}

public function testEmailArrayMessage()
Expand All @@ -170,7 +170,7 @@ public function testEmailArrayMessage()
EOT;

$ctrl = new DefaultController('default', Yii::$app->getModule('contactform'));
$this->assertContains($mail, $ctrl->generateMailMessage($model));
$this->assertStringContainsString($mail, $ctrl->generateMailMessage($model));
}

public function testGenerateMailAltBody()
Expand All @@ -183,7 +183,7 @@ public function testGenerateMailAltBody()
$ctrl = new DefaultController('default', Yii::$app->getModule('contactform'));
// in order to see the brs in tests we wrap with nl2br.
$altBody = $ctrl->generateMailAltBody($model);
$this->assertContains('Foo: bar, foo
$this->assertStringContainsString('Foo: bar, foo
Label for Field: Content', $altBody);
}

Expand All @@ -198,7 +198,7 @@ public function testGenerateMailAltBodyWithAttributesLAbels()
$ctrl = new DefaultController('default', $module);
// in order to see the brs in tests we wrap with nl2br.
$altBody = $ctrl->generateMailAltBody($model);
$this->assertContains('The Label: No', $altBody);
$this->assertStringContainsString('The Label: No', $altBody);
}

public function testModuleMailTemplateProperty()
Expand All @@ -210,7 +210,7 @@ public function testModuleMailTemplateProperty()
$module->mailTitle = '-';
$module->mailTemplate = '<p>foo{title}bar</p>';
$ctrl = new DefaultController('default', $module);
$this->assertSame('<p>foo-bar</p>', $ctrl->generateMailMessage($model));
$this->assertStringContainsString('<p>foo-bar</p>', $ctrl->generateMailMessage($model));
}

/**
Expand Down

0 comments on commit 8d4aaa3

Please sign in to comment.