Comments

shabana.navas created an issue. See original summary.

joshmiller’s picture

Status: Active » Needs work

If there are tests, I've not seen or heard of them.

The POS tests are probably a good starting point. I think we should, at the very least, test the following:

  • As an administrator, I can add single products to a purchase order.
  • As an administrator, I can add multiple products to a purchase order.
  • As an administrator, I can remove a product from a purchase order.
  • As an administrator, I can define a lot field and the incremental field widget will convert required units using stepping.
  • As an administrator, I can create an email with a PDF attached.

Nice to have tests would be:

  • Comprehensive Entity creation / update / delete tests.
  • Comprehensive tests that check the filters on various list admin interfaces
  • Comprehensive tests that confirm the notes/logging don't miss events.
  • Comprehensive tests that check that the configuration options display expected options (that none of the content types are missing from a drop down, for instance).

Ultimately, I want to see at least one test committed to the repo with issues opened for any remaining tests tagged as "low hanging fruit."

shabana.navas’s picture

Status: Needs work » Needs review
StatusFileSize
new4.4 KB

This is what I have currently. However, sadly, my tests are failing right at the point of enabling the modules. Seems like I'm not getting all the dependencies correct for commerce_pom to enable successfully. Any ideas??

joshmiller’s picture

Status: Needs review » Reviewed & tested by the community

Shabana++

+++ b/tests/commerce_pom.test
@@ -0,0 +1,143 @@
+    $term->pom_vendor_email[LANGUAGE_NONE][0]['value'] = 'snavas+vendor_1@acromediainc.com';
...
+        'vendor_email' => 'snavas+admin@acromediainc.com',

You might consider using a non-real email.

  • shabana.navas committed d956f33 on 7.x-1.x
    Issue #2878436 by shabana.navas, joshmiller: Write tests for POM
    
shabana.navas’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.