Merge pull request 5am-code#16 from 5am-code/dev

johguentner · web-flow · commit 536ef86b0fac · 2021-05-30T18:21:45.000+02:00
Dev
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,7 +1,3 @@
 # Changelog
 
-All notable changes to `laravel-notion-api` will be documented in this file
-
-## 1.0.0 - 201X-XX-XX
-
-- initial release
+Please check out the recent changelog in the [documentation](https://5amco.de/docs).
diff --git a/src/Endpoints/Pages.php b/src/Endpoints/Pages.php
@@ -39,7 +39,7 @@ public function find(string $pageId): Page
     public function create(): array
     {
         //toDo
-        throw new HandlingException('Not implemented');
+        throw new HandlingException( 'Not implemented');
     }
 
 
diff --git a/src/Notion.php b/src/Notion.php
@@ -50,6 +50,7 @@ class Notion
      * Notion constructor.
      * @param string|null $version
      * @param string|null $token
+     * @throws HandlingException
      */
     public function __construct(string $token, string $version = 'v1')
     {
@@ -58,16 +59,20 @@ public function __construct(string $token, string $version = 'v1')
         $this->validVersions = collect(['v1']);
 
         $this->setVersion($version);
+        $this->connect();
 
     }
 
     /**
      *
      * @return Notion
+     * @throws HandlingException
      */
     private function connect(): Notion
     {
-        $this->connection = Http::withToken($this->token);
+        $this->connection = Http
+            ::withHeaders($this->buildRequestHeader())
+            ->withToken($this->token);
         return $this;
     }
 
@@ -89,6 +94,7 @@ public function setVersion(string $version): Notion
      * Wrapper function to set version to v1.
      *
      * @return $this
+     * @throws HandlingException
      */
     public function v1(): Notion
     {
@@ -101,11 +107,11 @@ public function v1(): Notion
      *
      * @param string $token
      * @return Notion
+     * @deprecated for public usage; will be set to private in 0.4.0!
      */
     public function setToken(string $token): Notion
     {
         $this->token = $token;
-        $this->connect();
         return $this;
     }
 
@@ -195,7 +201,37 @@ public function getConnection(): ?PendingRequest
     public function checkValidVersion(string $version): void
     {
         if (!$this->validVersions->contains($version)) {
-            throw HandlingException::instance('invalid version for notion-api', ['invalidVersion' => $version]);
+            throw HandlingException::instance('Invalid version for Notion-API endpoint', ['invalidVersion' => $version]);
+        }
+    }
+
+    /**
+     * @return string[]
+     *
+     * @throws HandlingException
+     */
+    private function buildRequestHeader(): array
+    {
+        return [
+            'Notion-Version' => $this->mapVersionToHeaderVersion()
+        ];
+    }
+
+    /**
+     * Due to the inconsistency of the Notion API requiring a endpoint url
+     * with v* as well as a dated version in the request header, this method
+     * maps the given version (e.g. v1) to the version date Notion requires
+     * in the header (e.g. "2021-05-13").
+     * @return string
+     * @throws HandlingException
+     */
+    private function mapVersionToHeaderVersion(): string
+    {
+        switch ($this->version) {
+            case 'v1':
+                return '2021-05-13';
+            default:
+                throw new HandlingException('Invalid version.');
         }
     }
 }
diff --git a/tests/NotionApiTest.php b/tests/NotionApiTest.php
@@ -37,24 +37,9 @@ protected function getPackageAliases($app): array
         ];
     }
 
-
-    /** @test */
-    public function it_returns_notion_instance_with_set_token_and_connection()
-    {
-        $notion = new Notion('secret_*');
-        $notion->v1()->setToken('secret_*');
-
-        $this->assertInstanceOf(Notion::class, $notion);
-        $this->assertNotEmpty($notion->getConnection());
-    }
-
-
     /** @test */
-    public function it_throws_a_handling_exception_invalid_version()
+    public function it_asserts_true()
     {
-        $this->expectException(HandlingException::class);
-        $this->expectExceptionMessage('invalid version for notion-api');
-
-        new Notion('secret_*', 'v-does-not-exist');
+        $this->assertTrue(true);
     }
 }
diff --git a/tests/NotionTest.php b/tests/NotionTest.php
@@ -0,0 +1,30 @@
+<?php
+
+namespace FiveamCode\LaravelNotionApi\Tests;
+
+
+use FiveamCode\LaravelNotionApi\Exceptions\HandlingException;
+use FiveamCode\LaravelNotionApi\Notion;
+
+class NotionTest extends NotionApiTest
+{
+
+    /** @test */
+    public function it_returns_notion_instance_with_set_token_and_connection()
+    {
+        $notion = new Notion('secret_*');
+        $notion->v1()->setToken('secret_*');
+
+        $this->assertInstanceOf(Notion::class, $notion);
+        $this->assertNotEmpty($notion->getConnection());
+    }
+
+    /** @test */
+    public function it_throws_a_handling_exception_invalid_version()
+    {
+        $this->expectException(HandlingException::class);
+        $this->expectExceptionMessage('Invalid version for Notion-API endpoint');
+
+        new Notion('secret_*', 'v-does-not-exist');
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public function find(string $pageId): Page`
`39`	`39`	`public function create(): array`
`40`	`40`	`{`
`41`	`41`	`//toDo`
`42`		`- throw new HandlingException('Not implemented');`
	`42`	`+ throw new HandlingException( 'Not implemented');`
`43`	`43`	`}`
`44`	`44`
`45`	`45`