fix: multiple bug fixes and features

.github/workflows/actions.yml

@@ -0,0 +1,23 @@
+name: Run Tests for Backend
+
+on:
+  push:
+    branches: [ "main" ]
+  pull_request:
+    branches: [ "main" ]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+    - name: Checkout page source
+      uses: actions/checkout@v3
+
+    - name: Build the Docker image and run tests
+      run: docker compose -f "compose.test.yaml" up -d
+
+    - name: Print Logs for Container
+      run: |
+        export CONTAINER_ID=$(docker ps -aqf "name=ci") 
+        `docker logs -f $CONTAINER_ID` | grep -i "FAILED" && exit 1 || exit 0

compose.test.yaml

@@ -1,5 +1,5 @@
 services:
-  test:
+  test_ci:
     build:
       context: flask
       target: builder
@@ -10,9 +10,17 @@ services:
       - ./flask:/src
     depends_on:
       - tests_db
-    environment:
+    environment: # these are the test environment variables.
       - MONGO_DB_NAME=testregistry
       - MONGO_URI=mongodb://mongo:27017/testregistry
+      - MONGO_USER_NAME=testuser
+      - MONGO_PASSWORD=testpassword
+      - SALT=MYSALT
+      - HOST=localhost
+      - RESET_EMAIL=reset@localhost.com
+      - RESET_PASSWORD=reset
+      - SUDO_PASSWORD=fortran
+
   tests_db:
     image: mongo
 

compose.yaml

@@ -19,15 +19,31 @@ services:
     # flask requires SIGINT to stop gracefully
     # (default stop signal from Compose is SIGTERM)
     stop_signal: SIGINT
-    environment:
+    environment:  # these are the test environment variables.
       - FLASK_SERVER_PORT=9091
       - MONGO_DB_NAME=registry
       - MONGO_URI=mongodb://mongo:27017/fpmregistry
+      - SALT=MYSALT
+      - MONGO_DB_NAME=fpmregistry
+      - SUDO_PASSWORD=fortran
+      - MONGO_USER_NAME=fortran
+      - MONGO_PASSWORD=fortran
+      - HOST=localhost
+      - RESET_EMAIL=reset@localhost.com
+      - RESET_PASSWORD=reset
 
     volumes:
       - ./flask:/src
     depends_on:
       - mongo
 
+  validate_package:
+    build:
+      context: flask
+      dockerfile: validate_package.Dockerfile
+    stop_signal: SIGINT
+    volumes:
+      - ./flask:/src
+
   mongo:
     image: mongo

flask/packages.py

@@ -149,7 +149,7 @@ def upload():
 
     # Check whether license identifier is valid or not.
     if not is_valid_license_identifier(license_str=package_license):
-        return jsonify({"code": 400, "message": "Invalid license identifier"})
+        return jsonify({"code": 400, "message": f"Invalid license identifier {package_license}. Please check the SPDX license identifier list."})
 
     # Find the document that contains the upload token.
     namespace_doc = db.namespaces.find_one({"upload_tokens": {"$elemMatch": {"token": upload_token}}})

flask/requirements.txt

@@ -1,5 +1,5 @@
 pymongo
-flask
+flask==2.2.4
 python-dateutil
 python-dotenv
 bcrypt

flask/user.py

@@ -154,7 +154,6 @@ def delete_user():
 
 
 @app.route("/users/account", methods=["POST"])
-@swag_from("documentation/account.yaml", methods=["POST"])
 def account():
     uuid = request.form.get("uuid")
     if not uuid:
@@ -176,7 +175,6 @@ def account():
 
 
 @app.route("/users/admin", methods=["POST"])
-@swag_from("documentation/admin.yaml", methods=["POST"])
 def admin():
     uuid = request.form.get("uuid")
     if not uuid:
@@ -197,7 +195,6 @@ def admin():
 
 
 @app.route("/users/admin/transfer", methods=["POST"])
-@swag_from("documentation/admin.yaml", methods=["POST"])
 def transfer_account():
     uuid = request.form.get("uuid")
     if not uuid:

registry/package.json

@@ -19,6 +19,7 @@
     "react-redux": "^8.0.5",
     "react-router-dom": "^6.7.0",
     "react-scripts": "5.0.1",
+    "react-toastify": "^9.1.2",
     "redux": "^4.2.1",
     "redux-persist": "^6.0.0",
     "styled-components": "^5.3.6",

registry/src/components/packageItem.js

@@ -34,7 +34,7 @@ const PackageItem = ({ packageEntity }) => {
 
   return (
     <MDBListGroupItem id="list-item">
-      <Row>
+      <Row style={{ padding: "5px" }}>
         <Col md={1}>
           <Image
             src="https://fortran-lang.org/en/_static/fortran-logo-256x256.png"
@@ -43,25 +43,26 @@ const PackageItem = ({ packageEntity }) => {
             height={60}
           />
         </Col>
-        <Col md={4}>
-          <div>
-            <h5 id="list-item-package-name">
-              <a
-                href={`/packages/${packageEntity.namespace}/${packageEntity.name}`}
-                style={{ textDecoration: "none" }}
-              >
-                {packageEntity.name}
-              </a>
-            </h5>
-          </div>
+        <Col md={4} style={{ padding: "10px" }}>
+          <h5 id="list-item-package-name">
+            <a
+              href={`/packages/${packageEntity.namespace}/${packageEntity.name}`}
+              style={{ textDecoration: "none" }}
+            >
+              {packageEntity.name}
+            </a>
+          </h5>
           <h6 className="mb-2 text-muted">
             Namespace {packageEntity.namespace}
           </h6>
-          <label className="mb-2 text-muted">{packageEntity.description}</label>
+          <label className="mb-2 text-muted" style={{ fontSize: "18px" }}>
+            {packageEntity.description}
+          </label>
         </Col>
         <Col md={1} style={{ flex: 1, textAlign: "right" }}>
-          <h6>By {packageEntity.author}</h6>
-          <label>{formatDate(packageEntity.updatedAt)}</label>
+          <label style={{ fontSize: "16px" }}>
+            Last released {formatDate(packageEntity.updatedAt)}
+          </label>
         </Col>
       </Row>
     </MDBListGroupItem>

registry/src/pages/createNamespace.js

@@ -6,6 +6,8 @@ import Spinner from "react-bootstrap/Spinner";
 import Button from "react-bootstrap/Button";
 import Form from "react-bootstrap/Form";
 import { Card } from "react-bootstrap";
+import { ToastContainer, toast } from "react-toastify";
+import "react-toastify/dist/ReactToastify.css";
 
 const NamespaceForm = () => {
   const dispatch = useDispatch();
@@ -14,6 +16,26 @@ const NamespaceForm = () => {
   const isLoading = useSelector((state) => state.createNamespace.isLoading);
   const message = useSelector((state) => state.createNamespace.message);
   const statuscode = useSelector((state) => state.createNamespace.statuscode);
+  const toast_css = {
+    position: "top-center",
+    autoClose: 5000,
+    hideProgressBar: false,
+    closeOnClick: true,
+    pauseOnHover: true,
+    draggable: true,
+    progress: undefined,
+    theme: "light",
+  };
+
+  useEffect(() => {
+    if (statuscode === 200) {
+      toast.success(message, {
+        onClose: () => navigate("/manage/projects")
+      });
+    } else if (statuscode !== 0 && statuscode !== 200) {
+      toast.error(message, toast_css);
+    }
+  }, [statuscode]);
 
   useEffect(() => {
     if (uuid === null) {
@@ -40,6 +62,17 @@ const NamespaceForm = () => {
   return !isLoading ? (
     <Card id="create-namespace-card">
       <h3>Create a namespace</h3>
+      <ToastContainer
+        position="top-center"
+        autoClose={5000}
+        hideProgressBar={false}
+        newestOnTop={false}
+        closeOnClick
+        rtl={false}
+        pauseOnFocusLoss
+        pauseOnHover
+        theme="light"
+      />
       <Form onSubmit={handleSubmit}>
         <Form.Group
           className="mb-3"
@@ -82,11 +115,6 @@ const NamespaceForm = () => {
           Submit
         </Button>
       </Form>
-      {statuscode === 200 ? (
-        <p className="success">{message}</p>
-      ) : (
-        <p className="error">{message}</p>
-      )}
     </Card>
   ) : (
     <div className="d-flex justify-content-center">

registry/src/pages/namespace.js

@@ -50,7 +50,7 @@ const NamespacePage = () => {
               {" Namespace: " + namespace}
               </MDBIcon>
             <MDBIcon style={{ marginTop: "5px" }} far icon="calendar-alt">
-              {" Joined " + Date(dateJoined).slice(4, 15)}
+              {" Created " + Date(dateJoined).slice(4, 15)}
             </MDBIcon>
 
           </Row>