Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

チュートリアルで外税表示の税額が受け取れない #13

Closed
paper29squad opened this issue Jul 24, 2019 · 5 comments · Fixed by #14
Closed

チュートリアルで外税表示の税額が受け取れない #13

paper29squad opened this issue Jul 24, 2019 · 5 comments · Fixed by #14

Comments

@paper29squad
Copy link

paper29squad commented Jul 24, 2019

チュートリアルを試しました。
外税方式で表示するテストが失敗してしまいます。
理由は、taxが0を受け取っている。
内税方式で表示するテストは成功します。

> jinzo-ningen-test@1.0.0 test C:\bin\jinzo-ningen-test
> jest

  console.warn node_modules/jinzo-ningen/lib/login.js:18
    kintone login successful

 FAIL  ./index.test.js (8.237s)
  計算テスト
    入力と保存、表示
      新規作成画面での入力操作
        × 消費税額と合計額が外税で正しく計算できたこと (386ms)
        √ 消費税額と合計額が内税で正しく計算できたこと (315ms)

  ● 計算テスト › 入力と保存、表示 › 新規作成画面での入力操作 › 消費税額と合計額が外税で正しく計算できたこと

    expect(received).toEqual(expected) // deep equality

    Expected: 792
    Received: 0

      40 |         const total = await jz.getNumber(page, 'total');
      41 |
    > 42 |         expect(tax).toEqual(792);
         |                     ^
      43 |         expect(total).toEqual(9900);
      44 |       });
      45 |

      at Object.toEqual (index.test.js:42:21)

Test Suites: 1 failed, 1 total
Tests:       1 failed, 1 passed, 2 total
Snapshots:   0 total
Time:        8.518s, estimated 9s
Ran all test suites.
npm ERR! Test failed.  See above for more details.

環境
windows 10 pro
node v8.14.0
npm 6.9.0

jest-puppeteer.config.js
では
ignoreDefaultArgs: ['--disable-extensions'],
を追加している(chromeにポリシー適用されているため拡張が入ってしまうを防ぐ)

試したこと

  • calc.jsを
    event.record.tax.value = 1251;
    のように固定値にしても、外税方式は0を受け取り、内税方式は1251を受け取りエラーになる。
    常にtaxの額を受け取れないわけではない様子。
  • await jz.waitForMoment(page);
    をいくつか増やしてみる。
    (2485ms)のように時間はかかっているが、依然0を受け取ってエラーなる。
  • ヘッドレスモードにしてみるが変化なし。
@paper29squad
Copy link
Author

念のためと思って
index.test.js

        //const tax = await jz.getNumber(page, 'tax');
        const tax = 792;

とすると

    Expected: 9900
    Received: 0

      42 |
      43 |         expect(tax).toEqual(792);
    > 44 |         expect(total).toEqual(9900);
         |                       ^
      45 |       });

となりました。外税方式の場合、taxもtotalも値が取れないようです。

@paper29squad
Copy link
Author

原因がわかりました。
kintoneのフォーム設定画面で初期値が内税だと動きました。
外税だと受け取る値が0になりました。

 PASS  ./index.test.js (8.84s)
  計算テスト
    入力と保存、表示
      新規作成画面での入力操作
        √ 消費税額と合計額が外税で正しく計算できたこと (385ms)
        √ 消費税額と合計額が内税で正しく計算できたこと (315ms)

Test Suites: 1 passed, 1 total
Tests:       2 passed, 2 total
Snapshots:   0 total
Time:        8.994s, estimated 10s
Ran all test suites.

image

@latica-jp
Copy link
Collaborator

latica-jp commented Jul 24, 2019

@paper29squad ご報告をありがとうございました!
ご指摘の現象を確認しました。
モジュールの修正が必要かと思います。修正が完了しましたら、こちらでご報告の上、issue をクローズさせていただきます。
詳細にご報告いただき、たいへん助かりました。
引き続き、どうぞよろしくお願いいたします 🙇

@t-kojima
Copy link
Collaborator

t-kojima commented Aug 1, 2019

@paper29squad
こちら修正のうえ、v1.0.1としてリリース致しました。
ご報告ありがとうございました!

@paper29squad
Copy link
Author

確認しました。ありがとうございます。

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 a pull request may close this issue.

3 participants