Skip to content
This repository has been archived by the owner on Apr 1, 2024. It is now read-only.

Add testcode relate to current user #235

Merged
merged 9 commits into from
Dec 5, 2019

Conversation

vfa-tamhh
Copy link
Contributor

概要(Summary)

  • Fixed #xx

動作確認手順(Step for Confirmation)

Run the unit test.

  1. Clone source code.
  2. npm install
  3. npm test

Copy link
Contributor

@ncmbadmin ncmbadmin left a comment

Choose a reason for hiding this comment

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

ご対応ありがとうございます。
テストを確認させていただき、コメントがありますので、
ご確認いただけますでしょうか。

test/stub/2013-09-01_authentication_error_user.json Outdated Show resolved Hide resolved
test/users_test.js Outdated Show resolved Hide resolved
test/mbaas.yml Outdated Show resolved Hide resolved
test/users_test.js Show resolved Hide resolved
test/users_test.js Outdated Show resolved Hide resolved
Copy link
Contributor

@ncmbadmin ncmbadmin left a comment

Choose a reason for hiding this comment

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

ご対応ありがとうございます。細かいところですが、2点コメントがありますので、お手数ですが、ご確認お願い致します。

test/users_test.js Outdated Show resolved Hide resolved
test/users_test.js Outdated Show resolved Hide resolved
Copy link
Contributor

@ncmbadmin ncmbadmin left a comment

Choose a reason for hiding this comment

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

本件、ご対応ありがとうございます。
追加テストコードは問題ないことを確認しました。

782 passing

@ncmbadmin ncmbadmin merged commit 8c64561 into develop Dec 5, 2019
@ncmbadmin ncmbadmin deleted the add_testcode_relate_to_current_user branch December 5, 2019 04:43
@kobo0908 kobo0908 mentioned this pull request Dec 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants