switch to GitHub CI

.github/workflows/ci.yml

@@ -0,0 +1,36 @@
+name: CI
+
+on: [push, pull_request]
+
+env:
+    php-options: -C -d opcache.enable=0 -d zend.exception_ignore_args=0
+
+jobs:
+    tests:
+        runs-on: ${{ matrix.os }}
+        strategy:
+            matrix:
+                os: [ubuntu-latest]
+                php: ['8.0', '8.1', '8.2', '8.3']
+                sapi: ['php', 'php-cgi']
+
+            fail-fast: false
+
+        name: PHP ${{ matrix.php }}/${{ matrix.sapi }} tests on ${{ matrix.os }}
+        steps:
+            - uses: actions/checkout@v3
+            - uses: shivammathur/setup-php@v2
+              with:
+                  php-version: ${{ matrix.php }}
+                  coverage: none
+            - uses: actions/setup-java@v3
+              with:
+                  distribution: 'zulu'
+                  java-version: '17'
+            - uses: actions/setup-node@v3
+              with:
+                  node-version: '20'
+
+            - run: composer install --no-progress --prefer-dist
+            - run: vendor/bin/tester -C test/test.php -p ${{ matrix.sapi }} -s ${{ env.php-options }}
+            - run: cd .. && transit-php/bin/transit-format

.gitignore

@@ -1,4 +1,3 @@
 .DS_Store
 vendor/
 .idea/
-*.lock

README.md

@@ -1,4 +1,4 @@
-# transit-php [![CircleCI](https://circleci.com/gh/honzabrecka/transit-php/tree/master.svg?style=svg&circle-token=817187a55f77fb9bd7811bf5daa6e344885523fa)](https://circleci.com/gh/honzabrecka/transit-php/tree/master)
+# transit-php
 
 Transit is a data format and a set of libraries for conveying values between applications written in different languages. This library provides support for marshalling Transit data to/from PHP. Unlike the Java and Clojure implementations it relies on the non-streaming JSON parsing mechanism of the host PHP environment.
 

bin/roundtrip.php

@@ -1,5 +1,7 @@
 <?php
 
+error_reporting(E_ERROR | E_PARSE);
+
 require_once __DIR__ . '/../vendor/autoload.php';
 
 use transit\JSONReader;

composer.json

@@ -18,9 +18,9 @@
   },
   "require": {
     "php": ">=5.6.0",
-    "nette/utils": "2.5.*"
+    "nette/utils": "4.0.*"
   },
   "require-dev": {
-    "nette/tester": "2.0.*"
+    "nette/tester": "2.5.*"
   }
 }

src/transit/Transit.php

@@ -13,7 +13,6 @@
 use transit\handlers\URIHandler;
 use transit\handlers\UUIDHandler;
 use transit\handlers\CharHandler;
-use transit\handlers\TimestampHandler;
 use transit\handlers\ArbitraryPrecisionDecimalHandler;
 use transit\handlers\ArbitraryPrecisionIntegerHandler;
 
@@ -53,7 +52,6 @@ private function registerDefaultHandlers() {
         $this->registerHandler(new SymbolHandler());
         $this->registerHandler(new SetHandler());
         $this->registerHandler(new ListHandler());
-        $this->registerHandler(new TimestampHandler());
         $this->registerHandler(new DateTimeHandler());
         $this->registerHandler(new URIHandler());
         $this->registerHandler(new UUIDHandler());

src/transit/handlers/DateTimeHandler.php

@@ -13,11 +13,11 @@ public function type() {
     }
 
     public function representation($obj) {
-        return $obj->getTimestamp() * 1000;
+        return $obj->getTimestamp();
     }
 
     public function resolve($obj) {
-        return new \DateTime('@' . $obj/1000);
+        return new \DateTime('@' . $obj);
     }
 
 }

test/test.php

@@ -168,6 +168,10 @@ function r($input) {
   return t()->read($input);
 }
 
+function roundtrip($input) {
+  return w(r($input));
+}
+
 //-------------------------
 // write
 
@@ -414,8 +418,8 @@ function r($input) {
 //-------------------------
 // extensions
 
-Assert::equal('["~m482196050000"]', w([new DateTime('1985-04-12T23:20:50.52Z')]));
-Assert::equal((new DateTime('1985-04-12T23:20:50.52Z'))->getTimestamp(), r('["~m482196050000"]')[0]->getTimestamp());
+$timestamp = "[\"~#'\",\"~m1776669847291\"]";
+Assert::equal($timestamp, roundtrip($timestamp));
 
 Assert::equal('["~bYWJj"]', w([new Bytes('abc')]));
 Assert::equal([new Bytes('abc')], r('["~bYWJj"]'));