mirror of
https://github.com/silverstripe/silverstripe-behat-extension
synced 2024-10-22 17:05:32 +02:00
Merge branch '5.0' into 5
This commit is contained in:
commit
c35e9a440d
2
.github/workflows/ci.yml
vendored
2
.github/workflows/ci.yml
vendored
@ -9,3 +9,5 @@ jobs:
|
||||
ci:
|
||||
name: CI
|
||||
uses: silverstripe/gha-ci/.github/workflows/ci.yml@v1
|
||||
with:
|
||||
composer_require_extra: silverstripe/installer:4.13.x-dev
|
||||
|
@ -244,7 +244,7 @@ Feature: Do something with pages
|
||||
Scenario: View a page in the tree
|
||||
Given I am logged in with "ADMIN" permissions
|
||||
And I go to "/admin/pages"
|
||||
Then I should see "Page 1" in CMS Tree
|
||||
Then I should see "Page 1"
|
||||
```
|
||||
|
||||
* Fixtures are created where you defined them. If you want the fixtures to be created
|
||||
|
@ -294,22 +294,23 @@ class FixtureContext implements Context
|
||||
* @param string $null
|
||||
* @param TableNode $fieldsTable
|
||||
*/
|
||||
public function stepCreateRecordWithTable($type, $id, $null, TableNode $fieldsTable)
|
||||
public function stepCreateRecordWithTable($type, $id, TableNode $fieldsTable)
|
||||
{
|
||||
|
||||
$class = $this->convertTypeToClass($type);
|
||||
// TODO Support more than one record
|
||||
$fields = $this->convertFields($class, $fieldsTable->getRowsHash());
|
||||
$fields = $this->prepareFixture($class, $id, $fields);
|
||||
|
||||
// We should check if this fixture object already exists - if it does, we update it. If not, we create it
|
||||
if ($existingFixture = $this->fixtureFactory->get($class, $id)) {
|
||||
if ($existingFixture = $this->getFixtureFactory()->get($class, $id)) {
|
||||
// Merge existing data with new data, and create new object to replace existing object
|
||||
foreach ($fields as $k => $v) {
|
||||
$existingFixture->$k = $v;
|
||||
}
|
||||
$existingFixture->write();
|
||||
} else {
|
||||
$this->fixtureFactory->createObject($class, $id, $fields);
|
||||
$this->getFixtureFactory()->createObject($class, $id, $fields);
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user