3.1 Static Testing Basics: Reviews
3.1 Static Testing Basics: Reviews
3.1 Static Testing Basics: Reviews
-Initial review
-Individual review
ثببث
Explaining the scope, objectives, process,
Identifying review characteristics such as the review roles, and work products to the participants.
type with roles, activities, and checklists
Answering any questions that participants
Selecting the people to participate in the review and may have about the review.
allocating roles
Defining the entry and exit criteria for more formal Issue communication and analysis
review types (e.g., inspections)
Communicating identified potential
Checking that entry criteria are met. defects (e.g., in a review meeting)
Analyzing potential defects, assigning
ownership and status to them
Fixing and reporting Evaluating and documenting quality
characteristics
Creating defect reports Evaluating the review findings against the
Fixing defects found exit criteria to make a review decision
Communicating defects to the appropriate person or
team
Recording updated status of
Gathering metrics Individual review (i.e., individual preparation)
Checking that exit criteria are met
Accepting the work product when the exit criteria are Reviewing all or part of the work product
reached
Noting potential defects, recommendations, and
questions
Review Roles:
Management-
Reviewers-
Author-
Roles and responsibilities in a formal review:
Facilitator (moderator- )
Review Leader-
1- Author
Creates the work product under review
2- Management
Fixes defects in the work product under
review (if necessary)
Is responsible for review planning
4- Review leader
Decides who will be involved and organizes when and where it will take place.
5-Reviewers
Walkthrough
Main purposes: find defects, improve the software product, consider alternative implementations, evaluate conformance
to standards and specifications
Possible additional purposes: exchanging ideas about techniques or style variations, training of participants, achieving
consensus
Scribe is mandatory.
Possible further purposes: evaluating quality and building confidence in the work product, generating new ideas,
رييسيسسيسي
motivating and enabling authors to improve future work products, considering alternative implementations
Reviewers should be technical peers of the author, and technical experts in the same or other disciplines
Review meeting is optional, ideally led by a trained facilitator (typically not the author)
Inspection
Main purposes: detecting potential defects, evaluating quality and building confidence in the work product,
preventing future similar defects through author learning and root cause analysis.
Possible further purposes: motivating and enabling authors to improve future work products and the software
development process, achieving consensus.
Follows a defined process with formal documented outputs, based on rules and checklists.
Uses clearly defined roles, such as those specified in section 3.2.2 which are mandatory, and may include a dedicated
reader (who reads the work product aloud often paraphrase, i.e. describes it in own words, during the review meeting).
Reviewers are either peers of the author or experts in other disciplines that are relevant to the work product.
Scribe is mandatory.
Metrics are collected and used to improve the entire software development process, including the inspection process.
Peer Review
1-Ad hoc
In an ad hoc review, reviewers are provided with little or no guidance on how this task should be performed. Reviewers often read
the work product sequentially, identifying and documenting issues as they encounter them.
This technique is highly dependent on reviewer skills and may lead to many duplicate issues being reported by different reviewers.
2-Checklist-based
1-A checklist-based review is a systematic technique, whereby the reviewers detect issues based on checklists
that are distributed at review initiation (e.g., by the facilitator).
2- A review checklist consists of a set of questions based on potential defects, which may be derived from
experience.
3-Checklists should be specific to the type of work product under review and should be maintained regularly to
cover issue types missed in previous reviews.
4-The main advantage of the checklist-based technique is a systematic coverage of typical defect types. Care
should be taken not to simply follow the checklist in individual reviewing, but also to look for defects outside
the checklist.
5-Role-based
A role-based review is a technique in which the reviewers evaluate the work product from the
perspective of individual stakeholder roles.
Typical roles include specific end user types (experienced, inexperienced, senior, child, etc.),
and specific roles in the organization (user administrator, system administrator, performance
tester, etc.).
The same principles apply as in perspective-based reading because the roles are similar.
Each review has clear objectives, defined during review planning, and used as measurable exit criteria
Review types are applied which are suitable to achieve the objectives and are appropriate to the type and level
of software work products and participants
Any review techniques used, such as checklist-based or role-based reviewing, are suitable for effective defect
identification in the work product to be reviewed.
Any checklists used address the main risks and are up to date.
Large documents are written and reviewed in small chunks, so that quality control is exercised by providing
authors early and frequent feedback on defects.
Reviews are conducted on small chunks, so that reviewers do not lose concentration during individual review
and/or the review meeting.
The meeting is well-managed, so that participants consider it a valuable use of their time
The review is conducted in an atmosphere of trust; the outcome will not be used for the evaluation of the
participants.
Participants avoid body language and behaviors that might indicate boredom, exasperation, or hostility to
other participants