Skip to content

Commit f4f544c

Browse files
Merge pull request #18 from optimizely/aliabbasrizvi/fail_parse
Setting default values correctly
2 parents 8a9bd18 + f1f22e4 commit f4f544c

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

src/Optimizely/ProjectConfig.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -129,11 +129,11 @@ public function __construct($datafile, $logger, $errorHandler)
129129
$this->_projectId = $config['projectId'];
130130
$this->_revision = $config['revision'];
131131

132-
$groups = $config['groups'];
133-
$experiments = $config['experiments'];
134-
$events = $config['events'];
135-
$attributes = $config['attributes'];
136-
$audiences = $config['audiences'];
132+
$groups = $config['groups'] ?: [];
133+
$experiments = $config['experiments'] ?: [];
134+
$events = $config['events'] ?: [];
135+
$attributes = $config['attributes'] ?: [];
136+
$audiences = $config['audiences'] ?: [];
137137

138138
$this->_groupIdMap = ConfigParser::generateMap($groups, 'id', Group::class);
139139
$this->_experimentKeyMap = ConfigParser::generateMap($experiments, 'key', Experiment::class);

tests/OptimizelyTest.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -143,8 +143,6 @@ public function testValidateInputsInvalidFileJsonValidationNotSkipped()
143143
'Random datafile',
144144
false)
145145
);
146-
147-
$this->expectOutputRegex('/Provided "datafile" has invalid schema./');
148146
}
149147

150148
public function testValidateInputsInvalidFileJsonValidationSkipped()
@@ -156,8 +154,6 @@ public function testValidateInputsInvalidFileJsonValidationSkipped()
156154
$validateInputsMethod->invoke(new Optimizely('Random datafile', null, null, null, true),
157155
'Random datafile', true)
158156
);
159-
160-
$this->expectOutputRegex('/Provided "datafile" is in an invalid format./');
161157
}
162158

163159
public function testValidatePreconditionsExperimentNotRunning()

0 commit comments

Comments
 (0)