Migrar las ofertas de la aplicación de jobs vieja a la nueva (#544)

* Added pytest as a runner

* Added pytest as a runner on Makefile

* Added joboffers app

* Some fixes to the JobOffer model

* Added initial joboffer detail

* Added initial job offer add form

* Changed joboffer's urls to use their namespace

* Added some basic pytest's fixtures

* Updated JobOffer's factory to include all the needed fields

* Added joboffer creation test

* Added joboffer admin page

* Moved add button to bottom of the page (joboffers)

* Added search button in admin page (joboffers)

* Added joboffer edit view

* Changed joboffer's choices to their long version

* Added migration for company owner to UserCompanyProfile

* Added migration for Job model to JobOffer

* Adjusted pytest to don't execute migrations

* Added short description generation for jobs migration

* Added migrations for old jobs tags

* Added migration for joboffer's comments

* Changed job migration to mark also rejected job offers

* Added search field for UserCompanyProfile

* Added empty message for JobOfferHistoryView

* Changed joboffer model to accept NULL companies and the corresponding migrations

* Renamed migrations to avoid conflicts with previous branch

Co-authored-by: Juan Manuel Schillaci <[email protected]>
Co-authored-by: Ian Puchetti <[email protected]>

Author: 3 people

joboffers/migrations/0015_alter_joboffer_company.py

@@ -0,0 +1,20 @@
+# Generated by Django 3.2.12 on 2022-06-13 17:25
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('pycompanies', '0005_company_owner_to_usercompanyprofile'),
+        ('joboffers', '0014_alter_jobofferaccesslog_options'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='joboffer',
+            name='company',
+            field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.CASCADE, to='pycompanies.company', verbose_name='Empresa'),
+        ),
+    ]

joboffers/migrations/0016_migrate_old_jobs.py

@@ -0,0 +1,86 @@
+# Generated by Django 3.2.12 on 2022-06-13 17:30
+import html
+import re
+
+from datetime import timedelta
+
+from django.db import migrations
+from django.utils.timezone import now
+
+
+def get_short_description(description):
+    """
+    Deduce the short_description from a given html description string
+    """
+    description_stripped_tags = re.sub(r'<[^>]*>', ' ', description)
+    description_without_spaces = re.sub(r'\s+', ' ',
+                                        description_stripped_tags).strip()
+    description_unescaped = html.unescape(description_without_spaces)
+    return description_unescaped[:512]
+
+
+def forward(apps, schema_editor):
+    Job = apps.get_model('jobs', 'Job')
+    JobOffer = apps.get_model('joboffers', 'JobOffer')
+    UserCompanyProfile = apps.get_model('pycompanies', 'UserCompanyProfile')
+    User = apps.get_model('auth', 'User')
+
+    superuser = User.objects.filter(is_superuser=True).first()
+
+    for job in Job.objects.all():
+        if not job.email:
+            email = UserCompanyProfile.objects.filter(
+                company=job.company).first().user.email
+        else:
+            email = job.email
+
+        SENIORITY_TO_EXPERIENCE_MAP = {
+            '': '0',  # Tomo por defecto 0 a falta de otro valor mejor
+            'Trainee': '0',
+            'Junior': '2+',
+            'Semi Senior': '5+',
+            'Senior': '10+'
+        }
+
+        experience = SENIORITY_TO_EXPERIENCE_MAP[job.seniority]
+        remoteness = 'REMOTE' if job.remote_work else 'IN_OFFICE'
+
+        if job.is_active:
+            thirty_days_ago = now() - timedelta(days=30)
+
+            if job.modified < thirty_days_ago:
+                state = 'EXPIRED'
+            else:
+                state = 'ACTIVE'
+        elif job.jobinactivated_set.exists():
+            state = 'REJECTED'
+        else:
+            state = 'DEACTIVATED'
+
+        joboffer = JobOffer(id=job.id,
+                            title=job.title,
+                            company=job.company,
+                            location=job.location,
+                            contact_mail=email,
+                            experience=experience,
+                            remoteness=remoteness,
+                            hiring_type='OTHER',
+                            description=job.description,
+                            created_at=job.created,
+                            created_by=job.owner,
+                            modified_at=job.modified,
+                            modified_by=superuser,
+                            short_description=get_short_description(
+                                job.description),
+                            state=state)
+
+        joboffer.save()
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('joboffers', '0015_alter_joboffer_company'),
+    ]
+
+    operations = [migrations.RunPython(forward, migrations.RunPython.noop)]

joboffers/migrations/0017_migrate_old_jobs_tags.py

@@ -0,0 +1,31 @@
+# Generated by Django 3.2.12 on 2022-06-13 17:30
+
+from django.db import migrations
+
+
+def forward(apps, schema_editor):
+    ContentType = apps.get_model('contenttypes', 'ContentType')
+    TaggedItem = apps.get_model('taggit', 'TaggedItem')
+
+    job_content_type = ContentType.objects.get(app_label='jobs', model='job')
+    joboffer_content_type = ContentType.objects.get(app_label='joboffers',
+                                                    model='joboffer')
+
+    tagged_items = TaggedItem.objects.filter(content_type=job_content_type)
+
+    new_tags = [
+        TaggedItem(content_type=joboffer_content_type,
+                   object_id=tagged_item.object_id,
+                   tag=tagged_item.tag) for tagged_item in tagged_items
+    ]
+    for new_tag in new_tags:
+        new_tag.save()
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('joboffers', '0016_migrate_old_jobs'),
+    ]
+
+    operations = [migrations.RunPython(forward, migrations.RunPython.noop)]

joboffers/migrations/0018_migrate_old_job_comments.py

@@ -0,0 +1,32 @@
+# Generated by Django 3.2.12 on 2022-06-13 17:30
+
+from django.db import migrations
+
+
+def forward(apps, schema_editor):
+    JobInactivated = apps.get_model('jobs', 'jobinactivated')
+    JobOfferComment = apps.get_model('joboffers', 'joboffercomment')
+
+    User = apps.get_model('auth', 'User')
+
+    superuser = User.objects.filter(is_superuser=True).first()
+
+    for old_comment in JobInactivated.objects.filter(
+            job__company__isnull=False):
+        new_comment = JobOfferComment(
+            text=f"{old_comment.reason}: {old_comment.comment}",
+            comment_type='MODERATION',
+            created_at=old_comment.created,
+            created_by=superuser,
+            joboffer_id=old_comment.job_id)
+
+        new_comment.save()
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('joboffers', '0017_migrate_old_jobs_tags'),
+    ]
+
+    operations = [migrations.RunPython(forward, migrations.RunPython.noop)]

joboffers/models.py

@@ -88,6 +88,7 @@ class JobOffer(models.Model):
         'pycompanies.Company',
         verbose_name=_('Empresa'),
         on_delete=models.CASCADE,
+        null=True
     )
     location = models.CharField(max_length=100, blank=True, null=True, verbose_name=_('Lugar'))
     contact_mail = models.EmailField(

joboffers/templates/joboffers/joboffer_history.html

@@ -35,6 +35,8 @@ <h2>Historial de «{{ object.title }}» </h2>
               </div>
             </div>
           </div>
+          {% empty %}
+            <p>No hay registros históricos para esta oferta.</p>
           {% endfor %}
         </div>
         {% include 'joboffers/_paginator.html' with page_obj=page_obj %}

pycompanies/migrations/0004_alter_usercompanyprofile_user.py

@@ -0,0 +1,21 @@
+# Generated by Django 3.2.12 on 2022-06-07 19:19
+
+from django.conf import settings
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ('pycompanies', '0003_usercompanyprofile'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='usercompanyprofile',
+            name='user',
+            field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='company', to=settings.AUTH_USER_MODEL),
+        ),
+    ]

pycompanies/migrations/0005_company_owner_to_usercompanyprofile.py

@@ -0,0 +1,29 @@
+# Generated by Django 3.2.12 on 2022-06-07 13:57
+
+from django.db import migrations
+
+
+def forward(apps, schema_editor):
+    Company = apps.get_model('pycompanies', 'Company')
+    UserCompanyProfile = apps.get_model('pycompanies', 'UserCompanyProfile')
+
+    user_profiles = [
+        UserCompanyProfile(company_id=company_id, user_id=user_id)
+        for company_id, user_id in Company.objects.values_list('id', 'owner')
+    ]
+
+    UserCompanyProfile.objects.bulk_create(user_profiles)
+
+
+def backward(apps, schema_editor):
+    UserCompanyProfile = apps.get_model('pycompanies', 'UserCompanyProfile')
+    UserCompanyProfile.objects.all().delete()
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('pycompanies', '0004_alter_usercompanyprofile_user'),
+    ]
+
+    operations = [migrations.RunPython(forward, backward)]

pycompanies/models.py

@@ -17,9 +17,7 @@ class Company(TimeStampedModel):
     name = models.CharField('Nombre', max_length=255, unique=True)
     description = models.TextField('Descripción')
     photo = models.ImageField('Logo', upload_to='pycompanies/logos')
-    link = models.URLField('URL',
-                           help_text=_('Dirección web de la empresa')
-                           )
+    link = models.URLField('URL', help_text=_('Dirección web de la empresa'))
     rank = models.PositiveIntegerField(default=0)
 
     def get_absolute_url(self):
@@ -47,9 +45,9 @@ class UserCompanyProfile(models.Model):
     """Company data for a User."""
     objects = UserCompanyProfileManager()
 
-    user = models.OneToOneField(settings.AUTH_USER_MODEL,
-                                related_name='company',
-                                on_delete=models.CASCADE)
+    user = models.ForeignKey(settings.AUTH_USER_MODEL,
+                             related_name='company',
+                             on_delete=models.CASCADE)
     company = models.ForeignKey(Company,
                                 related_name='users',
                                 on_delete=models.CASCADE)
@@ -60,6 +58,7 @@ def __str__(self):
 
 #  SIGNALS
 
+
 @receiver(post_delete, sender=Company)
 def post_delete_user(sender, instance, *args, **kwargs):
     "Delete logo image after delete company"

pytest.ini

@@ -1,2 +1,3 @@
 [pytest]
 requests_mock_case_sensitive = True
+addopts = --no-migrations --reuse-db --create-db