RdKafka; Replace composer-modifying for testing with --ignore-platform-req argument

maartenderie · maartenderie · commit 59b04beae7bb · 2021-10-29T15:08:27.000+02:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -26,4 +26,6 @@ jobs:
         with: # ext-rdkafka not needed for tests, and a pain to install on CI;
           composer-options: "--ignore-platform-req=ext-rdkafka"
 
+      - run: sed -i 's/525568/16777471/' vendor/kwn/php-rdkafka-stubs/stubs/constants.php
+
       - run: vendor/bin/phpunit --exclude-group=functional
diff --git a/Tests/bootstrap.php b/Tests/bootstrap.php
@@ -12,8 +12,5 @@
     }
 }
 
-// Pretend that a high version was installed, opposed to 0.8.5 from kwn/php-rdkafka-stubs
-const RD_KAFKA_VERSION = 42424242;
-
 include_once $kafkaStubsDir.'/stubs/constants.php';
 include_once $kafkaStubsDir.'/stubs/functions.php';

Original file line number	Diff line number	Diff line change
`@@ -12,8 +12,5 @@`
`12`	`12`	`}`
`13`	`13`	`}`
`14`	`14`
`15`		`-// Pretend that a high version was installed, opposed to 0.8.5 from kwn/php-rdkafka-stubs`
`16`		`-const RD_KAFKA_VERSION = 42424242;`
`17`		`-`
`18`	`15`	`include_once $kafkaStubsDir.'/stubs/constants.php';`
`19`	`16`	`include_once $kafkaStubsDir.'/stubs/functions.php';`