From 16ba0275c0f2de68b12bb132b29cff6dcd50b39a Mon Sep 17 00:00:00 2001
From: deedydas <dd367@cornell.edu>
Date: Mon, 6 May 2013 10:30:24 -0700
Subject: [PATCH] Files Generating

Summary: Ref T2903

Test Plan: Files are being generating, albeit temporary.

Reviewers: epriestley, AnhNhan

Reviewed By: AnhNhan

CC: aran, Korvin, AnhNhan, chad

Maniphest Tasks: T2903

Differential Revision: https://secure.phabricator.com/D5801
---
 src/__phutil_library_map__.php                |  2 ++
 .../PhabricatorFileTestDataGenerator.php      | 28 +++++++++++++++++++
 .../image/PhabricatorLipsumMondrianArtist.php |  4 +--
 3 files changed, 32 insertions(+), 2 deletions(-)
 create mode 100644 src/applications/files/lipsum/PhabricatorFileTestDataGenerator.php

diff --git a/src/__phutil_library_map__.php b/src/__phutil_library_map__.php
index e05a1f087e..f798af18ac 100644
--- a/src/__phutil_library_map__.php
+++ b/src/__phutil_library_map__.php
@@ -990,6 +990,7 @@ phutil_register_library_map(array(
     'PhabricatorFileStorageEngine' => 'applications/files/engine/PhabricatorFileStorageEngine.php',
     'PhabricatorFileStorageEngineSelector' => 'applications/files/engineselector/PhabricatorFileStorageEngineSelector.php',
     'PhabricatorFileTestCase' => 'applications/files/storage/__tests__/PhabricatorFileTestCase.php',
+    'PhabricatorFileTestDataGenerator' => 'applications/files/lipsum/PhabricatorFileTestDataGenerator.php',
     'PhabricatorFileTransformController' => 'applications/files/controller/PhabricatorFileTransformController.php',
     'PhabricatorFileUploadController' => 'applications/files/controller/PhabricatorFileUploadController.php',
     'PhabricatorFileUploadException' => 'applications/files/exception/PhabricatorFileUploadException.php',
@@ -2724,6 +2725,7 @@ phutil_register_library_map(array(
     'PhabricatorFileStorageBlob' => 'PhabricatorFileDAO',
     'PhabricatorFileStorageConfigurationException' => 'Exception',
     'PhabricatorFileTestCase' => 'PhabricatorTestCase',
+    'PhabricatorFileTestDataGenerator' => 'PhabricatorTestDataGenerator',
     'PhabricatorFileTransformController' => 'PhabricatorFileController',
     'PhabricatorFileUploadController' => 'PhabricatorFileController',
     'PhabricatorFileUploadException' => 'Exception',
diff --git a/src/applications/files/lipsum/PhabricatorFileTestDataGenerator.php b/src/applications/files/lipsum/PhabricatorFileTestDataGenerator.php
new file mode 100644
index 0000000000..bc1ac4ca11
--- /dev/null
+++ b/src/applications/files/lipsum/PhabricatorFileTestDataGenerator.php
@@ -0,0 +1,28 @@
+<?php
+
+final class PhabricatorFileTestDataGenerator
+  extends PhabricatorTestDataGenerator {
+
+  public function generate() {
+    $authorPHID = $this->loadAuthorPHID();
+    $dimension = 1 << rand(5, 12);
+    $image = id(new PhabricatorLipsumMondrianArtist())
+      ->generate($dimension, $dimension);
+    $file = PhabricatorFile::newFromFileData(
+      $image,
+      array(
+        'name' => 'rand-'.rand(1000, 9999),
+      ));
+    $file->setAuthorPHID($authorPHID);
+    $file->setMimeType('image/jpeg');
+    return $file->save();
+  }
+
+  private function loadPhabrictorUserPHID() {
+    return $this->loadOneRandom("PhabricatorUser")->getPHID();
+  }
+
+  public function loadAuthorPHID() {
+    return $this->loadPhabrictorUserPHID();
+  }
+}
diff --git a/src/applications/lipsum/image/PhabricatorLipsumMondrianArtist.php b/src/applications/lipsum/image/PhabricatorLipsumMondrianArtist.php
index 39cb46a8a0..41cce9de89 100644
--- a/src/applications/lipsum/image/PhabricatorLipsumMondrianArtist.php
+++ b/src/applications/lipsum/image/PhabricatorLipsumMondrianArtist.php
@@ -30,8 +30,8 @@ final class PhabricatorLipsumMondrianArtist extends PhabricatorLipsumArtist {
 
     $fills = mt_rand(3, 8);
     for ($ii = 0; $ii < $fills; $ii++) {
-      $xp = mt_rand(0, $x);
-      $yp = mt_rand(0, $y);
+      $xp = mt_rand(0, $x - 1);
+      $yp = mt_rand(0, $y - 1);
       if (imagecolorat($image, $xp, $yp) != $c_white) {
         continue;
       }