[SPARK-5882][GraphX] Add a test for GraphLoader.edgeListFile

[maropu]

No description provided.

[srowen]

@maropu in general you need a JIRA for non-trivial changes. However is this just a part of SPARK-5450 / #4244 ?

[maropu]

oops, I missed it.
I'll fix it soon.

[maropu]

Fixed.

[ankurdave]

ok to test

[SparkQA]

Test build #27719 has finished for PR 4674 at commit cc5ac0b.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • class GraphLoaderSuite extends FunSuite with LocalSparkContext

[srowen]

@ankurdave are you progressing with this PR or should we close it?

[SparkQA]

Test build #41382 has finished for PR 4674 at commit cc5ac0b.

• This patch fails to build.
• This patch does not merge cleanly.
• This patch adds no public classes.

[andrewor14]

this is outdated. You now need to extend org.apache.spark.SparkFunSuite instead of FunSuite directly.

[andrewor14]

retest this please (this should fail scala style tests because of the issue I brought up)

[andrewor14]

@maropu would you mind bringing this up to date in master?
@ankurdave this looks like an OK change. Is it good to merge once it's rebased?

[maropu]

@andrewor14 okay

[maropu]

@andrewor14 @ankurdave Fixed. Also, could you merge #4399?

[andrewor14]

retest this please

[ankurdave]

@andrewor14 Yes, LGTM

[andrewor14]

retest this please

[SparkQA]

Test build #2236 has finished for PR 4674 at commit c1e2303.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):\n * class GraphLoaderSuite extends SparkFunSuite with LocalSparkContext\n

[andrewor14]

Merging into master