xfrocks / bdApi

[bd] Api add-on for XenForo
GNU General Public License v3.0
98 stars 52 forks source link

Xf2 tests ott #87

Closed luutruong closed 6 years ago

daohoangson commented 6 years ago

Đù, bug ùi.

On Mon, Oct 8, 2018, 09:20 Truong Luu notifications@github.com wrote:

@luutruong commented on this pull request.

In _files/tests/api/OttTest.php https://github.com/xfrocks/bdApi/pull/87#discussion_r223238857:

+

  • public function testWithGuest()
  • {
  • $this->requestActiveOttToken(0);
  • }
  • public function testWithUser()
  • {
  • $user = $this->dataUser();
  • $this->requestActiveOttToken($user['user_id']);
  • }
  • protected function requestActiveOttToken($userId)
  • {
  • $timestamp = time() + 5 * 30;
  • $accessToken = ($userId > 0) ? self::$accessToken : METHOD;

Cái đó e thấy đặt như nào cũng đc á. E random cũng chạy luôn :D

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/xfrocks/bdApi/pull/87#discussion_r223238857, or mute the thread https://github.com/notifications/unsubscribe-auth/AAOm6CToLP-La9pXMxNZLYxACuObc3U1ks5uirZrgaJpZM4XJf3D .

luutruong commented 6 years ago

Ủa vậy expect là nếu là guest thì bắt buộc cái accessToken='' ạ? Tại e chạy mấy ngày nay đều kiểu thế chắc nghĩ flow là zị :D

daohoangson commented 6 years ago

Uh, nó phải empty string huhu.

On Mon, Oct 8, 2018, 09:39 Truong Luu notifications@github.com wrote:

Ủa vậy expect là nếu là guest thì bắt buộc cái accessToken='' ạ? Tại e chạy mấy ngày nay đều kiểu thế chắc nghĩ flow là zị :D

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/xfrocks/bdApi/pull/87#issuecomment-427709473, or mute the thread https://github.com/notifications/unsubscribe-auth/AAOm6Pprq-Qk_T_QAGXUSxU4AaXjqHH_ks5uirr3gaJpZM4XJf3D .

daohoangson commented 6 years ago

Cái này okie chưa chế?

luutruong commented 6 years ago

Đã xong rồi ạ :D

daohoangson commented 6 years ago

Đang chạy ci test, chờ xem có pass không nha hihi.

daohoangson commented 6 years ago

Passed ùi. À mà di mấy cái test vô file zzz nha. File đó tập trung các tests không biết phân vào đâu =))

luutruong commented 6 years ago

Đã moved rồi nhé anh.