search

goqoo-on-kintone / jinzo-ningen

8 stars 0 forks source link

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

Closed paper29squad closed 5 years ago

paper29squad commented 5 years ago

チュートリアルを試しました。 外税方式で表示するテストが失敗してしまいます。 理由は、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にポリシー適用されているため拡張が入ってしまうを防ぐ）

試したこと

paper29squad commented 5 years ago

念のためと思って 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 commented 5 years ago

原因がわかりました。 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 commented 5 years ago

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

t-kojima commented 5 years ago

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

paper29squad commented 5 years ago

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