• はてなブックマーク
  • テクノロジー
  • idやclassを使ってテストを書くのは、もはやアンチパターンである - Qiita
  • Twitterでシェア
  • Facebookでシェア

idやclassを使ってテストを書くのは、もはやアンチパターンである - Qiita

テクノロジー カテゴリーの変更を依頼 記事元:qiita.com/akameco
適切な情報に変更
394users がブックマーク コメント 44
    共有

    • 記事へのコメント44件

    • 注目コメント
    • 新着コメント
    その他
    t_yamo
    解るけれど、テスト用マーカーでテスト用世界を構成してそれに対してテストしている感もあって、実世界に対するテストとして考えるとモヤモヤした部分もある。正しく写像しているかのテストが別にあればいいのかな。

    その他
    z1h4784
    classは変わっちゃうから後で困るよというのは分かるけど、idが変わったせいでテストが通らなかったらバグでいいんじゃない？デザインのためにidを変えるのって普通なのかな

    その他
    hiroshitoda
    気持ちはわかるんだけどアンチパターンは言い過ぎだし、こういう動機に至らせてしまうReactがそもそも開発プラットフォームとしては微妙な気がしている

    その他
    ryota-murakami
    参考記事の最後に「なぜidを使わないのか？」に対するQ&Aが掲載 されている。個人的にはidで管理しても結局'test-○○'のようなprefixを付与する事になる可能性が高いので、それだったらdata属性の方が筋が良い かなと。

    その他
    b_wa
    うちはjs用のclass名の命名規則があるので、デザインを変えてもそのclass名は変わらない。li.jxxx-item.xxx-itemみたいになっていて、前者がロジック用、後者がスタイル用。

    その他
    uehaj
    RNではtestidの方がポピュラーかな https://www.npmjs.com/package/babel-plugin-react-test-id

    その他
    mizchi
    facebookの中身をみても data-test-idついてるしな

    その他
    teppeis
    Facebook系はdata-testid使ってるっぽい / 目的と開発フロー次第かな

    その他
    luccafort
    個人的にはわかりやすいし除去できるなら別に問題ない気がするけどなーと思ってからブコメみたら「なるほど、確かに…」みたいな気持ちになった。複雑すぎるフロントエンドに問題があるのでは？という気持ち。

    その他
    ledsun
    テストコードの都合でプロダクトコードを変更するのは、原則悪手。idやクラス名はグローバルなんだから全置換すればええやん（sassやlessで入れ子にしていると面倒だな）。

    その他
    hujuu
    “data-testidというテストのための属性を使うことで、何をテストするのか非常に明確になりました。”

    その他
    dorapon2000
    “もしスタイルの変更のためにclass名を変えて、ロジックのテストが落ちるなんてことはあってはならないのです。”

    その他
    yarumato
    “data-testidというテストのための属性を使うことで、何をテストするのか明確になりました。テストからクラスを扱わないことで、自由にスタイルのためにクラス名を変更したり、いつでもCSS in JSに切り替えられます”

    その他
    tri-star
    data-testのようなテストでしか使用しない属性を定義してテストからはそれを参照する

    その他
    takatama
    分かりやすくて良い

    その他
    gfx
    せやな。

    その他
    kgr0210
    👀

    その他
    akahigeg
    id変えてテスト落ちたらテストが正常に機能してるってことなんでいいんじゃないかな。問題になるほど頻繁には変えないと思うし。classの方はテストに使ったことない

    その他
    honeybe
    お、せやな。

    その他
    luccafort
    luccafort 個人的にはわかりやすいし除去できるなら別に問題ない気がするけどなーと思ってからブコメみたら「なるほど、確かに…」みたいな気持ちになった。複雑すぎるフロントエンドに問題があるのでは？という気持ち。

    2017/11/29 リンク

    その他
    mas-higa
    "プロダクションではそのクラス名は残していますか" "まあ残しても問題はないですが"

    その他
    Ashizawa
    コンポーネント設計が悪いから名前が被るんじゃね？

    その他
    pmint
    テスト手法を勘違いしてるんだと思う。テスト失敗は悪だと思ってそう。/ DOM要素に詳しい型付けをしましょうという話。その型をここではテストに利用してる。data-testというよりdata-typeにして入力の検証にも使うべき。

    その他
    Nobkz
    idやclassを使って、テストを書くのは「アンチパターン」とか言ってるけどそもそも、これパターンなのか...というか、なぜそういうテストを書くのか意味が分からんマン。

    その他
    tamanugi
    「このidとclassはデザイナー側がつけたものか実装時につけられたものかどっちだ？」って迷うことがあるので、よさげに思える

    その他
    kawa-_-kawa
    なるほどと思いつつも、デザイナー側に要素追加・更新・削除時にtext-XX要素を重複・変更・削除しないように注意してもらう必要あるわけで、プログラマーは楽になるけどコストが他に移っただけに見える。

    その他
    deep_one
    「idやclassはスタイルのためのものなので」全く同意しない。特にidはバックエンドのデータ設計のレベルのものだろ。

    その他
    isaisstillalive
    idやclassを使わないのは賛成。フォームの名前に関しては実装と関連してるので *[name="email"] とかで探すようにしてる

    その他
    yuanying
    https://www.w3.org/TR/html5/dom.html 読んだけど、id や class はスタイルのためのものとはやっぱり思えない。単なる実装上の都合でしかものを言ってなくないか。

    その他
    takaesu
    フロントエンド テストのやり方

    その他
    enemyoffreedom
    できればテストやデザインに限らずに、合意なくidを変えないでほしいかな

    その他
    yamadar
    一理ある

    その他
    gnufrfr
    testって文字列が配信されんのやだな。data-でやるのは良さそう。って後で気づいたけど、この属性をプロダクション環境では出さないってできんの？すごいな。

    その他
    z1h4784
    z1h4784 classは変わっちゃうから後で困るよというのは分かるけど、idが変わったせいでテストが通らなかったらバグでいいんじゃない？デザインのためにidを変えるのって普通なのかな

    2017/11/20 リンク

    その他
    side_tana
    はい

    その他
    ustam
    どうでもいいけど定数は全て大文字で書くという慣例はもう無くなっちゃったの？

    その他
    w1234567
    seleniumテストとかIT業界の闇を煮詰めたものに触れるのはやめよう

    その他
    elephant1974
    「idやclassはスタイルのためのものなので、テストでそれを使うのはやめましょう」→言われてみればその通りだけど、ウチのはそういうレベルじゃないからなー。

    その他
    ledsun
    ledsun テストコードの都合でプロダクトコードを変更するのは、原則悪手。idやクラス名はグローバルなんだから全置換すればええやん（sassやlessで入れ子にしていると面倒だな）。

    2017/11/19 リンク

    その他
    t_yamo
    t_yamo 解るけれど、テスト用マーカーでテスト用世界を構成してそれに対してテストしている感もあって、実世界に対するテストとして考えるとモヤモヤした部分もある。正しく写像しているかのテストが別にあればいいのかな。

    2017/11/19 リンク

    その他

    注目コメント算出アルゴリズムの一部にLINEヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています

    アプリのスクリーンショット
    いまの話題をアプリでチェック！
    • バナー広告なし
    • ミュート機能あり
    • ダークモード搭載
    アプリをダウンロード

    関連記事

    idやclassを使ってテストを書くのは、もはやアンチパターンである - Qiita

    いきなり結論を書くと、idやclassはスタイルのためのものなので、テストでそれを使うのはやめましょう。...

    ブックマークしたユーザー

    すべてのユーザーの
    詳細を表示します

    同じサイトの新着

    同じサイトの新着をもっと読む

    いま人気の記事

    いま人気の記事をもっと読む

    いま人気の記事 - テクノロジー

    いま人気の記事 - テクノロジーをもっと読む

    新着記事 - テクノロジー

    新着記事 - テクノロジーをもっと読む

    同時期にブックマークされた記事

    いま人気の記事 - 企業メディア

    企業メディアをもっと読む

    はてなブックマーク

    公式Twitter

    はてなのサービス

    • App Storeからダウンロード
    • Google Playで手に入れよう
    Copyright © 2005-2025 Hatena. All Rights Reserved.