+ class CoprUniqueNameValidator2:

+     """

+     Validate that that Copr project name User gave us doesn't

+     cause duplicity.

+ 

+     This validator can only be used in CoprBaseForm descendants.

+ 

+     TODO: Replace all occurrences of CoprUniqueNameValidator with this.

+     """

+     usr_msg = "You already have"

+     grp_msg = "Group '{}' already has"

+ 

+     def __call__(self, form, field):

+         msg = self.usr_msg

+         if form.group:

+             msg = self.grp_msg.format(form.group.name)

+             existing = CoprsLogic.exists_for_group(

+                 form.group, field.data).first()

+         else:

+             existing = CoprsLogic.exists_for_user(

+                 form.user, field.data).first()

+ 

+         if existing:

+             msg = msg + " a project named \"{}\"".format(existing.name)

+             raise wtforms.ValidationError(msg)

+ 

+ 

+ class CoprBaseForm(FlaskForm):

+     """

+     All forms that modify Copr project should inherit from this.

+     """

+ 

+     def __init__(self, *args, user=None, group=None, **kwargs):

+         super().__init__(*args, **kwargs)

+         self.user = user

+         self.group = group

+ 

+ 

+ class CoprFedoraReviewForm(CoprBaseForm):

+     """

+     Simplified Copr form for FedoraReview-only project.

+     """

+     name = wtforms.StringField(

+         "Name",

+         validators=[

+             wtforms.validators.DataRequired(),

+             NameCharactersValidator(),

+             CoprUniqueNameValidator2(),

+             NameNotNumberValidator()

+         ])

+ 

+ 

+ {% extends "layout.html" %}

+ {% block title %}Add a Fedora Review Project{% endblock %}

+ {% block header %}Add a Fedora Review Project{% endblock %}

+ 

+ {% from "_helpers.html" import render_field %}

+ 

+ {% block breadcrumbs %}

+ <ol class="breadcrumb">

+   <li>

+     <a href="{{ url_for('coprs_ns.coprs_show') }}">Home</a>

+   </li>

+   <li>

+     <a href="{{ url_for('coprs_ns.coprs_by_user', username=g.user.name) }}">{{ g.user.name }}</a>

+   </li>

+   <li class="active">

+     New Fedora Review Project

+   </li>

+ </ol>

+ {% endblock %}

+ {% block body %}

+ 

+ <h1>Create a Fedora Review Project</h1>

+ <p>

+ You agree to build only <a href="https://docs.pagure.org/copr.copr/user_documentation.html#what-i-can-build-in-copr">allowed content</a> in Copr.

+ Check if your <a href="https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#Good_Licenses">license</a> is allowed.

+ </p>

+ <form action="{{ request.path }}" method=post>

+   <div class="panel panel-default">

+     <div class="panel-heading">

+       <h3 class="panel-title">Project information</h3>

+     </div>

+     <div class="panel-body">

+       {{ form.csrf_token }}

+       {{ render_field(form.name) }}

+     </div>

+   </div>

+   <input class="btn btn-primary" type="submit" value="Create Fedora Review Project">

+ </form>

+ 

+ {% endblock %}

+ <a href="{{url_for('coprs_ns.copr_add_fedora_review', username=g.user.name) }}"

+     class="btn button-new btn-secondary pull-right">

+   <span class="pficon pficon-add-circle-o"></span> Fedora Review </a>

+   <a href="{{url_for('coprs_ns.copr_add_fedora_review', username=g.user.name) }}"

+       class="btn button-new btn-secondary">

+     <span class="pficon pficon-add-circle-o"></span> Fedora Review </a>

+ 

+ @coprs_ns.route("/<username>/new-fedora-review/", methods=["GET", "POST"])

+ @login_required

+ def copr_add_fedora_review(username):  # pylint: disable=unused-argument

+     """ Show and process the simplified "Copr create" form for Fedora Review """

+     delete_after_days = 60

+ 

+     if username != flask.g.user.username:

+         # when accessed by accident

+         flask.flash("You can not add projects for '{}' user".format(username),

+                     "error")

+         return flask.redirect("/")

+ 

+     form = forms.CoprFedoraReviewForm(user=flask.g.user)

+     if flask.request.method == "POST":

+         if form.validate_on_submit():

+             copr = coprs_logic.CoprsLogic.add(

+                 flask.g.user,

+                 name=form.name.data,

+                 selected_chroots=["fedora-rawhide-x86_64"],

+                 description="Project was created only to execute Fedora Review "

+                             "tool for all builds.",

+                 instructions="You should ask the project owner before "

+                              "installing anything from this project.",

+                 unlisted_on_hp=True,

+                 delete_after_days=delete_after_days,

+                 follow_fedora_branching=False,

+                 fedora_review=True,

+             )

+             db.session.commit()

+             flask.flash(

+                 "New review project has been created successfully, will be removed "

+                 "after {} days (if not prolonged)".format(delete_after_days),

+                 "success")

+ 

+             return flask.redirect(

+                 helpers.copr_url('coprs_ns.copr_add_build', copr))

+ 

+         # validation problem

+         flask.flash("Error in project config", "error")

+ 

+     return flask.render_template("coprs/add_fedora_review.html", form=form)

+ 

+ 

+ def parse_web_form_error(html_text, variant="a"):

+ 

+     if variant == "a":

+         classes = "alert alert-danger"

+     elif variant == "b":

+         classes = "alert alert-danger alert-dismissable"

+ 

+     alerts = soup.findAll('div', class_=classes)

+     if variant == "a":

+         return [li.text for li in div.find_all("li")]

+     return div.text.strip()

+ from commands.create_chroot import create_chroot_function

+ 

+ from tests.request_test_api import parse_web_form_error

+ 

+     @pytest.mark.usefixtures("f_u1_ts_client", "f_mock_chroots", "f_db")

+     def test_fedora_review_project(self):

+         create_chroot_function(["fedora-rawhide-x86_64"])

+         route = "/coprs/{0}/new-fedora-review/".format(self.transaction_username)

+         resp = self.test_client.post(

+             route,

+             data={"name": "test-fedora-review"},

+             follow_redirects=False,

+         )

+         assert "user1/test-fedora-review/add_build" in resp.headers["Location"]

+         copr = self.models.Copr.query.get(1)

+         assert copr.full_name == "user1/test-fedora-review"

+         assert len(copr.active_chroots) == 1

+         assert copr.active_chroots[0].name == "fedora-rawhide-x86_64"

+         assert "Fedora Review tool" in copr.description

+         assert "You should ask the project owner" in copr.instructions

+         assert copr.fedora_review

+         assert copr.unlisted_on_hp

+ 

+         # re-request

+         resp = self.test_client.post(

+             route,

+             data={"name": "test-fedora-review"},

+             follow_redirects=True,

+         )

+         assert resp.status_code == 200  # error!

+         error = parse_web_form_error(resp.data, variant="b")

+         assert error == "Error in project config"

+ # too-few-public-methods

+ #     It's often useful to inherit from some (even library) class, and override

+ #     some static property or even just constructor.  This though makes PyLint

+ #     warn us if parent method doesn't provide the minimal amount of methods,

+ #     reported here: https://github.com/PyCQA/pylint/issues/4352

+ #     It's inconvenient to silence this per-class.

+ disable=import-error,too-few-public-methods