Skip to content

CI tests by js client#590

Merged
cipepser merged 19 commits intomasterfrom
feature/e2e-js
May 21, 2021
Merged

CI tests by js client#590
cipepser merged 19 commits intomasterfrom
feature/e2e-js

Conversation

@cipepser
Copy link
Copy Markdown
Collaborator

@cipepser cipepser commented May 19, 2021

Issueへのリンク

やったこと

  • CIに、jsでsodiumで暗号化したリクエストを生成→Anonifyへ送信
  • 状態遷移ができることまでを確認

やらないこと

  • なし

動作検証

  • CI上で状態遷移結果が正しく取得できるところまで確認

参考

  • なし

@cipepser cipepser marked this pull request as ready for review May 21, 2021 09:03
@cipepser cipepser requested a review from osuketh May 21, 2021 09:03

const encrypt = async ([pubkey, msg]) => {
await _sodium.ready;
const sodium = _sodium;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

こ、ここら辺は何をやっているです・・?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここもいらなかったです

@@ -0,0 +1,110 @@
const _sodium = require('libsodium-wrappers');
const axios = require('axios');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

axios インポート必要です?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

axiosBase にだけにしてみる説

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

axiosBase だけでいけた!
jsムジー

@@ -0,0 +1,110 @@
const _sodium = require('libsodium-wrappers');
const axios = require('axios');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

axiosBase にだけにしてみる説

@cipepser cipepser requested a review from osuketh May 21, 2021 10:35
@cipepser cipepser merged commit 96fd9f5 into master May 21, 2021
@cipepser cipepser deleted the feature/e2e-js branch May 21, 2021 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants