Skip to content

Commit e981d1b

Browse files
tests: update PortForward tests to check for correct associated rule values
1 parent 670a338 commit e981d1b

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

pfSense-pkg-RESTAPI/files/usr/local/pkg/RESTAPI/Tests/APIModelsPortForwardTestCase.inc

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@ class APIModelsPortForwardTestCase extends TestCase {
182182
/**
183183
* Checks that port forwards with associated firewall rules correctly create, update and delete the associated rule
184184
*/
185-
public function test_associated_firewall_rules() {
185+
public function test_associated_firewall_rules(): void {
186186
# Create a port forward to test with
187187
$port_forward = new PortForward(
188188
interface: 'wan',
@@ -208,8 +208,8 @@ class APIModelsPortForwardTestCase extends TestCase {
208208
$this->assert_equals($rule_q->first()->protocol->value, 'tcp');
209209
$this->assert_equals($rule_q->first()->source->value, '1.2.3.4');
210210
$this->assert_equals($rule_q->first()->source_port->value, '1234');
211-
$this->assert_equals($rule_q->first()->destination->value, '4.3.2.1');
212-
$this->assert_equals($rule_q->first()->destination_port->value, '4321');
211+
$this->assert_equals($rule_q->first()->destination->value, '127.0.0.1');
212+
$this->assert_equals($rule_q->first()->destination_port->value, '1234');
213213

214214
# Update the port forward and ensure the associated rule is updated
215215
$port_forward->from_representation(
@@ -228,8 +228,8 @@ class APIModelsPortForwardTestCase extends TestCase {
228228
$this->assert_equals($rule_q->first()->protocol->value, 'udp');
229229
$this->assert_equals($rule_q->first()->source->value, '4321::1');
230230
$this->assert_equals($rule_q->first()->source_port->value, '4321');
231-
$this->assert_equals($rule_q->first()->destination->value, '1234::1');
232-
$this->assert_equals($rule_q->first()->destination_port->value, '1234');
231+
$this->assert_equals($rule_q->first()->destination->value, 'fe80::1');
232+
$this->assert_equals($rule_q->first()->destination_port->value, '4321');
233233

234234
# Delete the port forward and ensure the associated rule is deleted
235235
$port_forward->delete();

0 commit comments

Comments
 (0)