開発者がテスターのコードを見直すことは良い考えですか?

Background : Currently working in a firm which
follows a practice where the QA’s e2e and API automation suites are
mandated to be reviewed before being pushed in a CI pipeline. It’s
a practice that is being followed across multiple projects within
the firm.

Issue : I’m currently working in a project
where I am the lone testing/QA member, along with 3 Front End
(Angular/React) and 3 Back end developers
(Ruby). The e2e suite uses Protractor,
Cucumber
with Typescript – loosely based on
this framework given by Ram Pasala. The API suites
use vanilla JavaScript and use Chakram as
the framework.

私のQAマネージャーとの私の毎月の議論では、パイプラインにプッシュされる前にコードを徹底的に見直す必要がありますが、
ProtractorTypescript会社の持つ全QAチームの Java
または C#で動作します。

今は、TS/JSの十分な専門知識を持つフロントエンド開発者の誰かがコードをレビューする必要があるという選択肢が残っています。
私は前にこのような状況に直面したことはありませんし、これが良いアイデアか悪いアイデアかどうかは分かりません。私は一般的にQAチームメンバーが一般的にコード(チーム内またはチーム外のQA内)をレビューすることを見てきましたが、標準的な方法ではないとも考えています。

私はこのアイデアに関して何らかの思考/賛否両論を聞くのが大好きです。これは良い/悪い練習ですか?

ベストアンサー

いくつかのガイドラインがあれば大丈夫です。
QAコードはソフトウェアです。それでもソフトウェア工学の原則は適用されるはずです。開発者の誰も分度器を知らないので、彼らは訓練されるべきです。最終的にアプリケーションにテストを追加できる可能性があります。

誰がレビューを書いているかについては、フロントエンドの開発者はおそらく良い選択です。彼らは、あなたがJavascriptコードを書くために学ぶことができる良い指針とヒントを持っているかもしれません。あなたのコードに目をつけておくのは良いことです。誰かがそれを呼び出すので、怠惰なコードを書くのを防ぎます。これにより、コードベースがよりきれいに保たれます。彼らがあなたに同意しないことを言うならば、あなたの誰かが何かを学ぶつもりです。彼らは自分の道が正しいかどうかを確信するか、テストについて何か教えています。誰もが市民である限り、開発者とQAの間で重要な関係を築いています。

私の会社でのQAコードレビューはほとんどすべてです。
Jenkinsの自動チェックで実施されるコードをマージするには、少なくとも1つの承認が必要です。最適なところでは、コードレビューは、使用するフレームワークを知っている人が行う必要があります。おそらくコードレビューを行っている開発者のためのトレーニングを提案するべきです。彼らは訓練を受けているため、テストスイートに書く/貢献するべきであるとも言いましょう。

返信を残す

メールアドレスが公開されることはありません。 * が付いている欄は必須項目です