weixin_40001805
weixin_40001805
2020-12-02 00:12

forth: Remove test on separators that are not spaces

Not all languages handle unicode well.

该提问来源于开源项目:exercism/problem-specifications

  • 点赞
  • 写回答
  • 关注问题
  • 收藏
  • 复制链接分享
  • 邀请回答

7条回答

  • weixin_39548968 weixin_39548968 5月前

    https://github.com/exercism/x-common/pull/529#discussion_r98614292

    点赞 评论 复制链接分享
  • weixin_39548968 weixin_39548968 5月前

    Why device control 3? That makes little sense. That does need to go away, regardless of what happens to the rest of the test case.

    点赞 评论 复制链接分享
  • weixin_39548968 weixin_39548968 5月前

    Don't forget test version.

    点赞 评论 复制链接分享
  • weixin_40001805 weixin_40001805 5月前

    OK, i'll remove the \u0013, and bump the version to 1.1.0.

    点赞 评论 复制链接分享
  • weixin_40001805 weixin_40001805 5月前

    For some reason though, I'm not keen on this test. I just felt it was a complication at the beginning, where I was keen to move on to the more interesting parts of the exercise. But I suppose others may feel differently.

    点赞 评论 复制链接分享
  • weixin_40007175 weixin_40007175 5月前

    👍 💯 On removing this test.

    • It doesn't test anything relevant to the problem.
    • the test name does not match the test contents.
    • space is the only separator used in the test cases and we've already (implicitly) tested for that.
    • This is not the first time someone has made this PR
    点赞 评论 复制链接分享
  • weixin_40001805 weixin_40001805 5月前

    OK, I'm convinced... I'll change this to delete the test altogether.

    点赞 评论 复制链接分享

相关推荐